Make strum optional

Author: bugadani

espflash/Cargo.toml

@@ -51,7 +51,7 @@ serde = { version = "1.0.196", features = ["derive"] }
 serialport = { version = "4.3.0", optional = true }
 sha2 = "0.10.8"
 slip-codec = { version = "0.4.0", optional = true }
-strum = { version = "0.26.1", features = ["derive"] }
+strum = { version = "0.26.1", features = ["derive"], optional = true }
 thiserror = "1.0.56"
 toml = "0.8.10"
 update-informer = { version = "1.1.0", optional = true }
@@ -83,7 +83,7 @@ cli = [
 ]
 
 # enables connecting to a device via serial port
-serialport = ["flashing", "dep:serialport", "dep:slip-codec", "dep:regex"]
+serialport = ["flashing", "dep:serialport", "dep:slip-codec", "dep:regex", "dep:strum"]
 
 # enables flash stubs and stub commands
 flashing = ["xmas-elf"]

espflash/src/error.rs

@@ -8,7 +8,8 @@ use std::{
 use miette::Diagnostic;
 #[cfg(feature = "serialport")]
 use slip_codec::SlipError;
-use strum::{FromRepr, VariantNames};
+#[cfg(feature = "serialpot")]
+use strum::FromRepr;
 use thiserror::Error;
 
 #[cfg(feature = "cli")]
@@ -184,6 +185,7 @@ pub enum Error {
     InvalidElf(#[from] ElfError),
 
     #[error("The bootloader returned an error")]
+    #[cfg(feature = "serialpot")]
     #[diagnostic(transparent)]
     RomError(#[from] RomError),
 
@@ -345,6 +347,7 @@ impl From<CommandType> for TimedOutCommand {
 #[derive(Clone, Copy, Debug, Default, Diagnostic, Error, FromRepr)]
 #[non_exhaustive]
 #[repr(u8)]
+#[cfg(feature = "serialpot")]
 pub enum RomErrorKind {
     #[error("Invalid message received")]
     #[diagnostic(code(espflash::rom::invalid_message))]
@@ -420,6 +423,7 @@ pub enum RomErrorKind {
     Other = 0xff,
 }
 
+#[cfg(feature = "serialpot")]
 impl From<u8> for RomErrorKind {
     fn from(raw: u8) -> Self {
         Self::from_repr(raw).unwrap_or_default()
@@ -429,13 +433,15 @@ impl From<u8> for RomErrorKind {
 /// An error originating from a device's ROM functionality
 #[derive(Clone, Copy, Debug, Diagnostic, Error)]
 #[error("Error while running {command} command")]
+#[cfg(feature = "serialpot")]
 #[non_exhaustive]
 pub struct RomError {
     command: CommandType,
     #[source]
     kind: RomErrorKind,
 }
 
+#[cfg(feature = "serialpot")]
 impl RomError {
     pub fn new(command: CommandType, kind: RomErrorKind) -> RomError {
         RomError { command, kind }