Skip to content

Commit

Permalink
Rollup merge of rust-lang#123500 - belovdv:remove-miri-jobserver-fixm…
Browse files Browse the repository at this point in the history
…e, r=RalfJung,oli-obk

Revert removing miri jobserver workaround

Reverts rust-lang#123469.

r? ``@ghost``
  • Loading branch information
GuillaumeGomez authored Apr 5, 2024
2 parents 5ceac29 + 8cfd199 commit 8a6f9a1
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/tools/miri/cargo-miri/src/phases.rs
Original file line number Diff line number Diff line change
Expand Up @@ -558,6 +558,13 @@ pub fn phase_runner(mut binary_args: impl Iterator<Item = String>, phase: Runner
// Set missing env vars. We prefer build-time env vars over run-time ones; see
// <https://github.com/rust-lang/miri/issues/1661> for the kind of issue that fixes.
for (name, val) in info.env {
// `CARGO_MAKEFLAGS` contains information about how to reach the jobserver, but by the time
// the program is being run, that jobserver no longer exists (cargo only runs the jobserver
// for the build portion of `cargo run`/`cargo test`). Hence we shouldn't forward this.
// Also see <https://github.com/rust-lang/rust/pull/113730>.
if name == "CARGO_MAKEFLAGS" {
continue;
}
if let Some(old_val) = env::var_os(&name) {
if old_val == val {
// This one did not actually change, no need to re-set it.
Expand Down

0 comments on commit 8a6f9a1

Please sign in to comment.