fix: process command line arguments passed to daemon BEFORE clap

Author: nik-rev

src/clipboard.rs

@@ -64,7 +64,7 @@ pub fn set_image(
             .arg(clipboard_buffer_path.path())
             .stdin(process::Stdio::null())
             .stdout(process::Stdio::null())
-            .stderr(process::Stdio::null())
+            .stderr(process::Stdio::inherit())
             .current_dir("/")
             .spawn()?;
     }

src/main.rs

@@ -11,17 +11,49 @@ use miette::IntoDiagnostic;
 const LOGO: &[u8; 64 * 64 * 4] = include_bytes!(concat!(env!("OUT_DIR"), "/logo.bin"));
 
 fn main() -> miette::Result<()> {
+    // On linux, a daemon is required to provide clipboard access even when
+    // the process dies.
+    //
+    // More info: <https://docs.rs/arboard/3.5.0/arboard/trait.SetExtLinux.html#tymethod.wait>
+    #[cfg(target_os = "linux")]
+    {
+        if std::env::args()
+            .nth(1)
+            .as_deref()
+            .is_some_and(|arg| arg == ferrishot::CLIPBOARD_DAEMON_ID)
+        {
+            ferrishot::run_clipboard_daemon().expect("Failed to run clipboard daemon");
+            return Ok(());
+        }
+    }
+
     // This will parse the command line arguments.
     //
     // Needs to come before the logging initialization because there
     // is an argument to change the verbosity
     LazyLock::force(&CLI);
 
+    // env_logger::Builder::new()
+    //     .format(|buf, record| {
+    //         writeln!(
+    //             buf,
+    //             "{}:{} {} [{}] - {}",
+    //             record.file().unwrap_or("unknown"),
+    //             record.line().unwrap_or(0),
+    //             chrono::Local::now().format("%Y-%m-%dT%H:%M:%S%.3f"),
+    //             record.level(),
+    //             record.args()
+    //         )
+    //     })
+    //     .target(env_logger::Target::Pipe(target))
+    //     .filter(None, log::LevelFilter::Error)
+    //     .init();
+
     // Initialize logging
     // TODO:
     // - log to file.
     // - add ways to increase logging with command line arguments. Currently you must use `RUST_LOG=info`
-    env_logger::builder().init();
+    // env_logger::builder().init();
 
     LazyLock::force(&ferrishot::CONFIG);
 
@@ -38,22 +70,6 @@ fn main() -> miette::Result<()> {
         return Ok(());
     }
 
-    // On linux, a daemon is required to provide clipboard access even when
-    // the process dies.
-    //
-    // More info: <https://docs.rs/arboard/3.5.0/arboard/trait.SetExtLinux.html#tymethod.wait>
-    #[cfg(target_os = "linux")]
-    {
-        if std::env::args()
-            .nth(1)
-            .as_deref()
-            .is_some_and(|arg| arg == ferrishot::CLIPBOARD_DAEMON_ID)
-        {
-            ferrishot::run_clipboard_daemon().expect("Failed to run clipboard daemon");
-            return Ok(());
-        }
-    }
-
     iced::application(App::default, App::update, App::view)
         .window(iced::window::Settings {
             level: iced::window::Level::Normal,