refactor: use target_os specific error enum definition

Signed-off-by: Reuben Miller <reuben.d.miller@gmail.com>

Author: reubenmiller

crates/core/tedge_watchdog/src/dummy_watchdog.rs

@@ -1,7 +1,11 @@
 use std::path::PathBuf;
 
 pub async fn start_watchdog(_config_dir: PathBuf) -> Result<(), anyhow::Error> {
-    Err(anyhow::Error::from(
-        crate::error::WatchdogError::WatchdogNotAvailable,
-    ))
+    Err(anyhow::Error::from(WatchdogError::WatchdogNotAvailable))
+}
+
+#[derive(Debug, thiserror::Error)]
+pub enum WatchdogError {
+    #[error("The watchdog is not available on this platform")]
+    WatchdogNotAvailable,
 }

crates/core/tedge_watchdog/src/error.rs

@@ -1,17 +1,11 @@
-use time::error::Parse;
-
 use mqtt_channel::MqttError;
-
 use tedge_config::CertificateError;
 use tedge_config::ConfigSettingError;
 use tedge_config::TEdgeConfigError;
+use time::error::Parse;
 
 #[derive(Debug, thiserror::Error)]
 pub enum WatchdogError {
-    #[cfg(not(target_os = "linux"))]
-    #[error("The watchdog is not available on this platform")]
-    WatchdogNotAvailable,
-
     #[error("MQTT receiver closed")]
     ChannelClosed,
 

crates/core/tedge_watchdog/src/lib.rs

@@ -2,13 +2,13 @@ use std::path::PathBuf;
 use tedge_config::system_services::*;
 use tedge_config::DEFAULT_TEDGE_CONFIG_PATH;
 
-mod error;
-
 // on linux, we use systemd
 #[cfg(target_os = "linux")]
 mod systemd_watchdog;
 #[cfg(target_os = "linux")]
 use systemd_watchdog as watchdog;
+#[cfg(target_os = "linux")]
+mod error;
 
 // on non-linux, we do nothing for now
 #[cfg(not(target_os = "linux"))]