about summary refs log tree commit diff
path: root/clippy_dev
diff options
context:
space:
mode:
authorAlex Macleod <alex@macleod.io>2022-10-25 11:32:49 +0000
committerAlex Macleod <alex@macleod.io>2022-10-25 11:32:49 +0000
commitbd8369089c38c190533b41ce76b073f5478f17d4 (patch)
tree33522089e2b61338dc3d14d0427e0575873454d5 /clippy_dev
parent5b09d4e1f7082aff024faf27263f78e7fc7190a2 (diff)
downloadrust-bd8369089c38c190533b41ce76b073f5478f17d4.tar.gz
rust-bd8369089c38c190533b41ce76b073f5478f17d4.zip
Track `clippy.toml` and `Cargo.toml` in `file_depinfo`
Causes cargo to re-run clippy when those paths are modified

Also tracks the path to `clippy-driver` in debug mode to remove the
workarounds in `cargo dev lint` and `lintcheck`
Diffstat (limited to 'clippy_dev')
-rw-r--r--clippy_dev/Cargo.toml1
-rw-r--r--clippy_dev/src/lint.rs8
2 files changed, 0 insertions, 9 deletions
diff --git a/clippy_dev/Cargo.toml b/clippy_dev/Cargo.toml
index 2ac3b4fe2ed..510c7e852af 100644
--- a/clippy_dev/Cargo.toml
+++ b/clippy_dev/Cargo.toml
@@ -10,7 +10,6 @@ indoc = "1.0"
 itertools = "0.10.1"
 opener = "0.5"
 shell-escape = "0.1"
-tempfile = "3.2"
 walkdir = "2.3"
 
 [features]
diff --git a/clippy_dev/src/lint.rs b/clippy_dev/src/lint.rs
index 71005449b4d..aafd0f71a59 100644
--- a/clippy_dev/src/lint.rs
+++ b/clippy_dev/src/lint.rs
@@ -36,20 +36,12 @@ pub fn run<'a>(path: &str, args: impl Iterator<Item = &'a String>) {
     } else {
         exit_if_err(Command::new("cargo").arg("build").status());
 
-        // Run in a tempdir as changes to clippy do not retrigger linting
-        let target = tempfile::Builder::new()
-            .prefix("clippy")
-            .tempdir()
-            .expect("failed to create tempdir");
-
         let status = Command::new(cargo_clippy_path())
             .arg("clippy")
             .args(args)
             .current_dir(path)
-            .env("CARGO_TARGET_DIR", target.as_ref())
             .status();
 
-        target.close().expect("failed to remove tempdir");
         exit_if_err(status);
     }
 }