Rollup merge of #111969 - clubby789:clean-dry-run, r=Mark-Simulacrum

matthiaskrgr · web-flow · commit 95b909a119d5 · 2023-06-03T20:38:11.000+02:00
bootstrap: Make `clean` respect `dry-run`

I noticed `clean_default` was getting run twice as the `DryRun::SelfCheck` flag is ignored
diff --git a/src/bootstrap/clean.rs b/src/bootstrap/clean.rs
@@ -85,6 +85,10 @@ clean_crate_tree! {
 }
 
 fn clean_default(build: &Build, all: bool) {
+    if build.config.dry_run() {
+        return;
+    }
+
     rm_rf("tmp".as_ref());
 
     if all {

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,10 @@ clean_crate_tree! {`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`fn clean_default(build: &Build, all: bool) {`
	`88`	`+ if build.config.dry_run() {`
	`89`	`+ return;`
	`90`	`+ }`
	`91`	`+`
`88`	`92`	`rm_rf("tmp".as_ref());`
`89`	`93`
`90`	`94`	`if all {`