fix: Normalize the target path

Author: linyihai

crates/cargo-util/src/paths.rs

@@ -75,6 +75,8 @@ pub fn dylib_path() -> Vec<PathBuf> {
 
 /// Normalize a path, removing things like `.` and `..`.
 ///
+/// CAUTION: This expects the path to be absolute.
+///
 /// CAUTION: This does not resolve symlinks (unlike
 /// [`std::fs::canonicalize`]). This may cause incorrect or surprising
 /// behavior at times. This should be used carefully. Unfortunately,

src/cargo/ops/vendor.rs

@@ -4,6 +4,7 @@ use crate::ops;
 use crate::sources::path::PathSource;
 use crate::sources::CRATES_IO_REGISTRY;
 use crate::util::cache_lock::CacheLockMode;
+use crate::util::toml::AbsolutePathTomlTarget;
 use crate::util::{try_canonicalize, CargoResult, GlobalContext};
 use anyhow::{bail, Context as _};
 use cargo_util::{paths, Sha256};
@@ -412,9 +413,13 @@ fn prepare_for_vendor(
 ) -> CargoResult<Package> {
     let contents = me.manifest().contents();
     let document = me.manifest().document();
+    let abs_targets =
+        crate::util::toml::absolute_normalized_targets(me.manifest().normalized_toml(), me.root())?;
     let original_toml = prepare_toml_for_vendor(
         me.manifest().normalized_toml().clone(),
         packaged_files,
+        &abs_targets,
+        me.root(),
         gctx,
     )?;
     let normalized_toml = original_toml.clone();
@@ -428,10 +433,12 @@ fn prepare_for_vendor(
         document.clone(),
         original_toml,
         normalized_toml,
+        &abs_targets,
         features,
         workspace_config,
         source_id,
         me.manifest_path(),
+        me.root(),
         gctx,
         &mut warnings,
         &mut errors,
@@ -443,14 +450,19 @@ fn prepare_for_vendor(
 fn prepare_toml_for_vendor(
     mut me: cargo_util_schemas::manifest::TomlManifest,
     packaged_files: &[PathBuf],
+    abs_targets: &AbsolutePathTomlTarget,
+    package_root: &Path,
     gctx: &GlobalContext,
 ) -> CargoResult<cargo_util_schemas::manifest::TomlManifest> {
     let package = me
         .package
         .as_mut()
         .expect("venedored manifests must have packages");
-    if let Some(cargo_util_schemas::manifest::StringOrBool::String(path)) = &package.build {
-        let path = paths::normalize_path(Path::new(path));
+    if let Some(path) = &abs_targets.build {
+        let path = path
+            .strip_prefix(package_root)
+            .expect("path shoud be under the package root")
+            .to_path_buf();
         let included = packaged_files.contains(&path);
         let build = if included {
             let path = path
@@ -469,37 +481,42 @@ fn prepare_toml_for_vendor(
         package.build = Some(build);
     }
 
-    let lib = if let Some(target) = &me.lib {
+    let lib = if let Some(target) = &abs_targets.lib {
         crate::util::toml::prepare_target_for_publish(
             target,
             Some(packaged_files),
+            package_root,
             "library",
             gctx,
         )?
     } else {
         None
     };
     let bin = crate::util::toml::prepare_targets_for_publish(
-        me.bin.as_ref(),
+        abs_targets.bin.as_ref(),
         Some(packaged_files),
+        package_root,
         "binary",
         gctx,
     )?;
     let example = crate::util::toml::prepare_targets_for_publish(
-        me.example.as_ref(),
+        abs_targets.example.as_ref(),
         Some(packaged_files),
+        package_root,
         "example",
         gctx,
     )?;
     let test = crate::util::toml::prepare_targets_for_publish(
-        me.test.as_ref(),
+        abs_targets.test.as_ref(),
         Some(packaged_files),
+        package_root,
         "test",
         gctx,
     )?;
     let bench = crate::util::toml::prepare_targets_for_publish(
-        me.bench.as_ref(),
+        abs_targets.bench.as_ref(),
         Some(packaged_files),
+        package_root,
         "benchmark",
         gctx,
     )?;