From d2daf8ecf6a3d0d8782ac25e369491ab10a40f34 Mon Sep 17 00:00:00 2001 From: Shotaro Yamada Date: Tue, 8 Oct 2019 14:20:24 +0900 Subject: [PATCH] Revert "Workaround cargo bug on Windows" This reverts commit 248251b3b26c5f21e2c2f5bfa5d85cc17b13fc05. This reverts commit 20b73514395bb442fb16ce913b6c2b3965ff3156. --- clippy_dev/Cargo.toml | 2 -- clippy_dev/src/fmt.rs | 15 +++------------ 2 files changed, 3 insertions(+), 14 deletions(-) diff --git a/clippy_dev/Cargo.toml b/clippy_dev/Cargo.toml index de56e4b05..e2e946d06 100644 --- a/clippy_dev/Cargo.toml +++ b/clippy_dev/Cargo.toml @@ -11,5 +11,3 @@ regex = "1" lazy_static = "1.0" shell-escape = "0.1" walkdir = "2" -# FIXME: remove this once cargo issue #7475 is fixed -home = "0.5" diff --git a/clippy_dev/src/fmt.rs b/clippy_dev/src/fmt.rs index 3c90aa4b5..69cd046a8 100644 --- a/clippy_dev/src/fmt.rs +++ b/clippy_dev/src/fmt.rs @@ -137,13 +137,13 @@ fn cargo_fmt(context: &FmtContext, path: &Path) -> Result { args.push("--"); args.push("--check"); } - let success = exec(context, &bin_path("cargo"), path, &args)?; + let success = exec(context, "cargo", path, &args)?; Ok(success) } fn rustfmt_test(context: &FmtContext) -> Result<(), CliError> { - let program = bin_path("rustfmt"); + let program = "rustfmt"; let dir = std::env::current_dir()?; let args = &["+nightly", "--version"]; @@ -170,7 +170,7 @@ fn rustfmt(context: &FmtContext, path: &Path) -> Result { if context.check { args.push("--check".as_ref()); } - let success = exec(context, &bin_path("rustfmt"), std::env::current_dir()?, &args)?; + let success = exec(context, "rustfmt", std::env::current_dir()?, &args)?; if !success { eprintln!("rustfmt failed on {}", path.display()); } @@ -195,12 +195,3 @@ fn project_root() -> Result { Err(CliError::ProjectRootNotFound) } - -// Workaround for https://github.com/rust-lang/cargo/issues/7475. -// FIXME: replace `&bin_path("command")` with `"command"` once the issue is fixed -fn bin_path(bin: &str) -> String { - let mut p = home::cargo_home().unwrap(); - p.push("bin"); - p.push(bin); - p.display().to_string() -}