bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Fix x test --stage 1 ui-fulldeps on macOS (until the next beta bump)

Browse source 
"stage 1" for fulldeps means "compile with stage 0, link against stage 1".
But this code wanted to switch on the compiler that's building, not the
compiler that's being tested. Fix the check.

Previously, it would fail with a warning about linker-messages:
```
--- stderr -------------------------------
warning[E0602]: unknown lint: `linker_messages`
   |
   = note: requested on the command line with `-A linker_messages`
   = note: `#[warn(unknown_lints)]` on by default
```
This commit is contained in:
jyn 2025-02-13 09:57:05 -05:00
parent 4a43094662
commit f7a03d075f
1 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

14
src/bootstrap/src/core/build_steps/test.rs
View file

@ -1840,6 +1840,14 @@ NOTE: if you're sure you want to do this, please open an issue as to why. In the
} }
} }
// FIXME(136096): on macOS, we get linker warnings about duplicate `-lm` flags.
// NOTE: `stage > 1` here because `test --stage 1 ui-fulldeps` is a hack that compiles
// with stage 0, but links the tests against stage 1.
// cfg(bootstrap) - remove only the `stage > 1` check, leave everything else.
if suite == "ui-fulldeps" && compiler.stage > 1 && target.ends_with("darwin") {
flags.push("-Alinker_messages".into());
}
let mut hostflags = flags.clone(); let mut hostflags = flags.clone();
hostflags.push(format!("-Lnative={}", builder.test_helpers_out(compiler.host).display())); hostflags.push(format!("-Lnative={}", builder.test_helpers_out(compiler.host).display()));
hostflags.extend(linker_flags(builder, compiler.host, LldThreads::No)); hostflags.extend(linker_flags(builder, compiler.host, LldThreads::No));
@ -1847,12 +1855,6 @@ NOTE: if you're sure you want to do this, please open an issue as to why. In the
let mut targetflags = flags; let mut targetflags = flags;
targetflags.push(format!("-Lnative={}", builder.test_helpers_out(target).display())); targetflags.push(format!("-Lnative={}", builder.test_helpers_out(target).display()));
// FIXME: on macOS, we get linker warnings about duplicate `-lm` flags. We should investigate why this happens.
if suite == "ui-fulldeps" && target.ends_with("darwin") {
hostflags.push("-Alinker_messages".into());
targetflags.push("-Alinker_messages".into());
}
for flag in hostflags { for flag in hostflags {
cmd.arg("--host-rustcflags").arg(flag); cmd.arg("--host-rustcflags").arg(flag);
} }