Only call default steps once, not once for each PathSet

Running steps multiple times defeats the whole point of #96501,
since lint messages will be duplicated.

diff --git a/src/bootstrap/builder.rs b/src/bootstrap/builder.rs
index 62b5416cee8af011605dd4ff05011826ad4f9161..d58829721cc9a90810bef11c2bea8ed603219837 100644 (file)
--- a/src/bootstrap/builder.rs
+++ b/src/bootstrap/builder.rs
@@ -304,9 +304,7 @@ fn run(v: &[StepDescription], builder: &Builder<'_>, paths: &[PathBuf]) {
         if paths.is_empty() || builder.config.include_default_paths {
             for (desc, should_run) in v.iter().zip(&should_runs) {
                 if desc.default && should_run.is_really_default() {
-                    for pathset in &should_run.paths {
-                        desc.maybe_run(builder, vec![pathset.clone()]);
-                    }
+                    desc.maybe_run(builder, should_run.paths.iter().cloned().collect());
                 }
             }
         }