Skip to content

Commit ee8e0bc

Browse files
committed
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.
1 parent 0e1a6fb commit ee8e0bc

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/bootstrap/builder.rs

+1-3
Original file line numberDiff line numberDiff line change
@@ -304,9 +304,7 @@ impl StepDescription {
304304
if paths.is_empty() || builder.config.include_default_paths {
305305
for (desc, should_run) in v.iter().zip(&should_runs) {
306306
if desc.default && should_run.is_really_default() {
307-
for pathset in &should_run.paths {
308-
desc.maybe_run(builder, vec![pathset.clone()]);
309-
}
307+
desc.maybe_run(builder, should_run.paths.iter().cloned().collect());
310308
}
311309
}
312310
}

0 commit comments

Comments
 (0)