Dividing the Actions PR - Linux

Author: steel-bucket

.github/workflows/build_mac.yml

@@ -22,7 +22,6 @@ libc = "0.2" # For setpgid and signal constants
 tracing-subscriber = { version = "0.3", features = ["env-filter", "fmt"] } # For logging setup
 tempfile = "3" # For creating temporary directories in examples/tests
 anyhow = "1.0" # For simple error handling in examples
-windows-sys = { version = "0.59.0", features = ["Win32", "Win32_System", "Win32_System_Threading", "Win32_Foundation"] }
 # Dependency on the library itself (needed for building examples within the workspace)
 # This line might not be strictly necessary if cargo automatically detects it,
 # but explicitly listing it can sometimes help.

.github/workflows/build_windows.yml

@@ -1,9 +1,7 @@
 // examples/git_clone_kernel.rs
-#[cfg(unix)]
-use std::os::unix::process::ExitStatusExt;
-#[cfg(windows)]
-use std::os::windows::process::ExitStatusExt;
+
 use command_timeout::{run_command_with_timeout, CommandError, CommandOutput};
+use std::os::unix::process::ExitStatusExt;
 use std::path::PathBuf; // <<< Import PathBuf
 use std::process::{Command, ExitStatus};
 use std::time::Duration;
@@ -36,11 +34,11 @@ async fn main() -> Result<(), anyhow::Error> {
     let min_timeout = Duration::from_secs(10);
     let max_timeout = Duration::from_secs(60 * 60 * 24 * 7); // 1 week (effectively infinite)
     let activity_timeout = Duration::from_secs(60 * 5); // 5 minutes
-    // -----------------------------
+                                                        // -----------------------------
 
     // Create a temporary directory builder
     let temp_dir_builder = Builder::new().prefix("kernel_clone_persistent").tempdir()?; // Use a different prefix
-    // --- CHANGE: Keep the path instead of the TempDir object ---
+                                                                                        // --- CHANGE: Keep the path instead of the TempDir object ---
     let clone_target_path_buf: PathBuf = temp_dir_builder.into_path();
     // --- END CHANGE ---
     let clone_target_path_str = clone_target_path_buf.to_str().unwrap_or("."); // Use "." as fallback if path invalid unicode
@@ -124,14 +122,9 @@ fn handle_command_output(output: CommandOutput) {
                 warn!("Exit Code: {}", code);
             }
             // signal() is now available because ExitStatusExt is in scope
-            #[cfg(unix)]
             if let Some(signal) = status.signal() {
                 warn!("Terminated by Signal: {}", signal);
             }
-            #[cfg(windows)]
-            if let Some(code) = status.code() {
-                warn!("Exit Code: {}", code);
-            }
         }
     } else {
         warn!("Exit Status: None (Killed by timeout, status unavailable?)");