Fix run_make on Windows

Author: encounter

Cargo.lock

@@ -52,7 +52,6 @@ bindings = [
 build = [
     "dep:notify",
     "dep:notify-debouncer-full",
-    "dep:path-slash",
     "dep:reqwest",
     "dep:self_update",
     "dep:shell-escape",
@@ -186,7 +185,6 @@ tempfile = { version = "3.15", optional = true }
 time = { version = "0.3", optional = true }
 
 [target.'cfg(windows)'.dependencies]
-path-slash = { version = "0.2", optional = true }
 winapi = { version = "0.3", optional = true }
 
 # For Linux static binaries, use rustls

objdiff-core/Cargo.toml

@@ -2,7 +2,7 @@ pub mod watcher;
 
 use std::process::Command;
 
-use typed_path::Utf8PlatformPathBuf;
+use typed_path::{Utf8PlatformPathBuf, Utf8UnixPath};
 
 pub struct BuildStatus {
     pub success: bool,
@@ -31,7 +31,7 @@ pub struct BuildConfig {
     pub selected_wsl_distro: Option<String>,
 }
 
-pub fn run_make(config: &BuildConfig, arg: &str) -> BuildStatus {
+pub fn run_make(config: &BuildConfig, arg: &Utf8UnixPath) -> BuildStatus {
     let Some(cwd) = &config.project_dir else {
         return BuildStatus {
             success: false,
@@ -51,7 +51,6 @@ pub fn run_make(config: &BuildConfig, arg: &str) -> BuildStatus {
     let mut command = {
         use std::os::windows::process::CommandExt;
 
-        use path_slash::PathExt;
         let mut command = if config.selected_wsl_distro.is_some() {
             Command::new("wsl")
         } else {
@@ -61,8 +60,8 @@ pub fn run_make(config: &BuildConfig, arg: &str) -> BuildStatus {
             // Strip distro root prefix \\wsl.localhost\{distro}
             let wsl_path_prefix = format!("\\\\wsl.localhost\\{}", distro);
             let cwd = match cwd.strip_prefix(wsl_path_prefix) {
-                Ok(new_cwd) => format!("/{}", new_cwd.to_slash_lossy().as_ref()),
-                Err(_) => cwd.to_string_lossy().to_string(),
+                Ok(new_cwd) => Utf8UnixPath::new("/").join(new_cwd.with_unix_encoding()),
+                Err(_) => cwd.with_unix_encoding(),
             };
 
             command
@@ -73,9 +72,9 @@ pub fn run_make(config: &BuildConfig, arg: &str) -> BuildStatus {
                 .arg("--")
                 .arg(make)
                 .args(make_args)
-                .arg(arg.to_slash_lossy().as_ref());
+                .arg(arg);
         } else {
-            command.current_dir(cwd).args(make_args).arg(arg.to_slash_lossy().as_ref());
+            command.current_dir(cwd).args(make_args).arg(arg);
         }
         command.creation_flags(winapi::um::winbase::CREATE_NO_WINDOW);
         command

objdiff-core/src/build/mod.rs

@@ -1,6 +1,6 @@
 use std::{sync::mpsc::Receiver, task::Waker};
 
-use anyhow::{anyhow, Error, Result};
+use anyhow::{bail, Error, Result};
 use time::OffsetDateTime;
 use typed_path::Utf8PlatformPathBuf;
 
@@ -43,14 +43,20 @@ fn run_build(
             .as_ref()
             .ok_or_else(|| Error::msg("Missing project dir"))?;
         if let Some(target_path) = &config.target_path {
-            target_path_rel = Some(target_path.strip_prefix(project_dir).map_err(|_| {
-                anyhow!("Target path '{}' doesn't begin with '{}'", target_path, project_dir)
-            })?);
+            target_path_rel = match target_path.strip_prefix(project_dir) {
+                Ok(p) => Some(p.with_unix_encoding()),
+                Err(_) => {
+                    bail!("Target path '{}' doesn't begin with '{}'", target_path, project_dir);
+                }
+            };
         }
         if let Some(base_path) = &config.base_path {
-            base_path_rel = Some(base_path.strip_prefix(project_dir).map_err(|_| {
-                anyhow!("Base path '{}' doesn't begin with '{}'", base_path, project_dir)
-            })?);
+            base_path_rel = match base_path.strip_prefix(project_dir) {
+                Ok(p) => Some(p.with_unix_encoding()),
+                Err(_) => {
+                    bail!("Base path '{}' doesn't begin with '{}'", base_path, project_dir);
+                }
+            };
         };
     }