cargo/config: Don't canonicalize joined [env] relative paths

Cargo doesn't do this either, and canonicalization requires the path to exist which it does not have to.
rust-mobile · May 15, 2023 · ff3cc28 · ff3cc28
1 parent 2f3aefb
commit ff3cc28
Showing 1 changed file with 7 additions and 10 deletions.
diff --git a/xbuild/src/cargo/config.rs b/xbuild/src/cargo/config.rs
@@ -111,16 +111,13 @@ impl EnvOption {
                 value,
                 relative: true,
                 force: _,
-            } => {
-                let value = config_parent.as_ref().join(value);
-                let value = dunce::canonicalize(&value)
-                    .with_context(|| format!("Failed to canonicalize `{}`", value.display()))?;
-                value
-                    .into_os_string()
-                    .into_string()
-                    .map_err(VarError::NotUnicode)?
-                    .into()
-            }
+            } => config_parent
+                .as_ref()
+                .join(value)
+                .into_os_string()
+                .into_string()
+                .map_err(VarError::NotUnicode)?
+                .into(),
             Self::String(value) | Self::Value { value, .. } => value.into(),
         })
     }