Normalize paths, just not commands

Author: jyn514

src/cmd/mod.rs

@@ -369,7 +369,7 @@ impl<'w, 'pl> Command<'w, 'pl> {
                 capture,
             )
         } else {
-            let (binary, managed_by_rustwide) = match self.binary {
+            let (mut binary, managed_by_rustwide) = match self.binary {
                 Binary::Global(path) => (path, false),
                 Binary::ManagedByRustwide(path) => (
                     self.workspace
@@ -381,6 +381,13 @@ impl<'w, 'pl> Command<'w, 'pl> {
                 ),
                 Binary::__NonExaustive => panic!("do not create __NonExaustive variants manually"),
             };
+            // This is the relative path to a command,
+            // so we need to normalize it or it won't be found if we change directories
+            // NOTE: if components.count() == 1, then this is a command that will be looked up in $PATH,
+            // so we shouldn't normalize it.
+            if binary.is_relative() && binary.components().count() > 1 {
+                binary = crate::utils::normalize_path(&binary);
+            }
             let mut cmd = AsyncCommand::new(&binary);
 
             cmd.args(&self.args);