print the full destination path when no track duplicates

Signed-off-by: hi-rustin <rustin.liu@gmail.com>
rust-lang · Aug 31, 2021 · 5da9bed · 5da9bed
1 parent c227565
commit 5da9bed
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/cargo/ops/cargo_install.rs b/src/cargo/ops/cargo_install.rs
@@ -258,7 +258,13 @@ impl<'cfg, 'a> InstallablePackage<'cfg, 'a> {
         if !self.force && !duplicates.is_empty() {
             let mut msg: Vec<String> = duplicates
                 .iter()
-                .map(|(name, _)| format!("binary `{}` already exists in destination", name))
+                .map(|(name, _)| {
+                    format!(
+                        "binary `{}` already exists in destination `{}`",
+                        name,
+                        dst.join(name).to_string_lossy()
+                    )
+                })
                 .collect();
             msg.push("Add --force to overwrite".to_string());
             bail!("{}", msg.join("\n"));

diff --git a/tests/testsuite/install_upgrade.rs b/tests/testsuite/install_upgrade.rs
@@ -694,7 +694,7 @@ fn no_track() {
         .with_stderr(
             "\
 [UPDATING] `[..]` index
-[ERROR] binary `foo[EXE]` already exists in destination
+[ERROR] binary `foo[EXE]` already exists in destination `[..]/.cargo/bin/foo`
 Add --force to overwrite
 ",
         )