Skip to content

Commit 8a6f9a1

Browse files
Rollup merge of #123500 - belovdv:remove-miri-jobserver-fixme, r=RalfJung,oli-obk
Revert removing miri jobserver workaround Reverts #123469. r? ``@ghost``
2 parents 5ceac29 + 8cfd199 commit 8a6f9a1

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/tools/miri/cargo-miri/src/phases.rs

+7
Original file line numberDiff line numberDiff line change
@@ -558,6 +558,13 @@ pub fn phase_runner(mut binary_args: impl Iterator<Item = String>, phase: Runner
558558
// Set missing env vars. We prefer build-time env vars over run-time ones; see
559559
// <https://github.com/rust-lang/miri/issues/1661> for the kind of issue that fixes.
560560
for (name, val) in info.env {
561+
// `CARGO_MAKEFLAGS` contains information about how to reach the jobserver, but by the time
562+
// the program is being run, that jobserver no longer exists (cargo only runs the jobserver
563+
// for the build portion of `cargo run`/`cargo test`). Hence we shouldn't forward this.
564+
// Also see <https://github.com/rust-lang/rust/pull/113730>.
565+
if name == "CARGO_MAKEFLAGS" {
566+
continue;
567+
}
561568
if let Some(old_val) = env::var_os(&name) {
562569
if old_val == val {
563570
// This one did not actually change, no need to re-set it.

0 commit comments

Comments
 (0)