Adress review comments

Author: blyxyas

lintcheck/src/config.rs

@@ -13,6 +13,7 @@ pub(crate) struct LintcheckConfig {
         value_name = "N",
         default_value_t = 0,
         default_value_if("perf", "true", Some("1")), // Limit jobs to 1 when benchmarking
+        conflicts_with("perf"),
         required = false,
         hide_default_value = true
     )]

lintcheck/src/main.rs

@@ -131,7 +131,6 @@ impl Crate {
                 "--",
                 "cargo",
             ]);
-            cmd.env("CARGO_PROFILE_RELEASE_DEBUG", "true");
         } else {
             cmd = Command::new("cargo");
         }
@@ -254,17 +253,25 @@ fn normalize_diag(
 
 /// Builds clippy inside the repo to make sure we have a clippy executable we can use.
 fn build_clippy(release_build: bool) -> String {
-    let output = Command::new("cargo")
+    let mut build_cmd = Command::new("cargo");
+    build_cmd
         .args([
             "run",
             "--bin=clippy-driver",
             if release_build { "-r" } else { "" },
             "--",
             "--version",
-        ])
+        ]);
+    
+    if release_build {
+        build_cmd.env("CARGO_PROFILE_RELEASE_DEBUG", "true");
+    }
+    
+    let output = build_cmd
         .stderr(Stdio::inherit())
         .output()
         .unwrap();
+
     if !output.status.success() {
         eprintln!("Error: Failed to compile Clippy!");
         std::process::exit(1);
@@ -286,13 +293,6 @@ fn main() {
 
     let config = LintcheckConfig::new();
 
-    if config.perf && config.max_jobs != 1 {
-        eprintln!(
-            "Lintcheck's --perf flag must be triggered only with 1 job,\nremove either the --jobs/-j flag or the --perf flag"
-        );
-        return;
-    }
-
     match config.subcommand {
         Some(Commands::Diff { old, new, truncate }) => json::diff(&old, &new, truncate),
         Some(Commands::Popular { output, number }) => popular_crates::fetch(output, number).unwrap(),