Skip to content

Commit 7e8b1b6

Browse files
committed
Auto merge of #123517 - GuillaumeGomez:rollup-eys3jfp, r=GuillaumeGomez
Rollup of 8 pull requests Successful merges: - #121419 (Add aarch64-apple-visionos and aarch64-apple-visionos-sim tier 3 targets) - #123159 (Fix target-cpu fpu features on Arm R/M-profile) - #123487 (CFI: Restore typeid_for_instance default behavior) - #123500 (Revert removing miri jobserver workaround) - #123505 (Revert "Use OS thread name by default") - #123509 (Add jieyouxu to compiler review rotation and as a reviewer for `tests/run-make`, `src/tools/run-make-support` and `src/tools/compiletest`) - #123514 (Fix typo in `compiler/rustc_middle/src/traits/solve/inspect.rs`) - #123515 (Use `include` command to reduce code duplication) r? `@ghost` `@rustbot` modify labels: rollup
2 parents 171f99f + 8c5486c commit 7e8b1b6

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

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)