Skip to content

Commit

Permalink
Simplify command executions for plain source tarballs
Browse files Browse the repository at this point in the history
If we're in dry run mode, the command will return an empty string, so we can just execute it.
  • Loading branch information
Kobzol committed Jul 12, 2024
1 parent 0cab962 commit 1089de4
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions src/bootstrap/src/core/build_steps/dist.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1060,11 +1060,7 @@ impl Step for PlainSourceTarball {
cmd.arg("--sync").arg(manifest_path);
}

let config = if !builder.config.dry_run() {
cmd.capture().run(builder).stdout()
} else {
String::new()
};
let config = cmd.capture().run(builder).stdout();

let cargo_config_dir = plain_dst_src.join(".cargo");
builder.create_dir(&cargo_config_dir);
Expand Down

0 comments on commit 1089de4

Please sign in to comment.