Skip to content

coretest in miri: fix using unstable libtest features#153437

Open
RalfJung wants to merge 2 commits intorust-lang:mainfrom
RalfJung:miri-libtest-unstable
Open

coretest in miri: fix using unstable libtest features#153437
RalfJung wants to merge 2 commits intorust-lang:mainfrom
RalfJung:miri-libtest-unstable

Conversation

@RalfJung
Copy link
Member

@RalfJung RalfJung commented Mar 5, 2026

Alternative (IMO preferable) to #153369. Also reverts that PR.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Mar 5, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 5, 2026

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: bootstrap
  • bootstrap expanded to 6 candidates
  • Random selection from Mark-Simulacrum, clubby789, jieyouxu

@RalfJung
Copy link
Member Author

RalfJung commented Mar 5, 2026

@bors try jobs=x86_64-gnu-aux

@rust-bors

This comment has been minimized.

rust-bors bot pushed a commit that referenced this pull request Mar 5, 2026
coretest in miri: fix using unstable libtest features


try-job: x86_64-gnu-aux
@RalfJung RalfJung force-pushed the miri-libtest-unstable branch from de493e4 to 07a902a Compare March 5, 2026 12:40
@rustbot
Copy link
Collaborator

rustbot commented Mar 5, 2026

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 5, 2026

☀️ Try build successful (CI)
Build commit: dec0b01 (dec0b01a659033e24d58dc8a50312191a40fd290, parent: f8704be04fe1150527fc2cf21dd44327f0fe87fb)

@cuviper
Copy link
Member

cuviper commented Mar 5, 2026

I think for the failure we saw on beta, you'll have to also set that here:
https://github.com/RalfJung/rust/blob/07a902a84445c32fa8106fb2b2ee44c38d43b4c3/src/ci/docker/host-x86_64/x86_64-gnu-miri/check-miri.sh

I was reproducing that failure on main by setting rust.channel = "beta" and running the command:

$ ./x miri --stage 2 library/core --test-args notest

... which is one of the last commands in that check script, with no MIRIFLAGS at all yet.
(It does work if I add the env-forward for that command though, as expected.)

@cuviper
Copy link
Member

cuviper commented Mar 5, 2026

It might be easier to have bootstrap set/append that to MIRIFLAGS itself, rather than trying to find all the places we call it in CI.

@RalfJung
Copy link
Member Author

RalfJung commented Mar 5, 2026

Oh right, we're also invoking ./x miri there...

(It does work if I add the env-forward for that command though, as expected.)

Thanks for confirming!

This reverts commit 30d7ed4.
It should not be needed any more.
@RalfJung RalfJung force-pushed the miri-libtest-unstable branch from 07a902a to e3b206c Compare March 5, 2026 21:09
@rustbot rustbot added the A-testsuite Area: The testsuite used to check the correctness of rustc label Mar 5, 2026
@RalfJung
Copy link
Member Author

RalfJung commented Mar 5, 2026

It might be easier to have bootstrap set/append that to MIRIFLAGS itself, rather than trying to find all the places we call it in CI.

Fair point. I tried that in the latest revision that I just pushed.

@RalfJung RalfJung force-pushed the miri-libtest-unstable branch from e3b206c to 6b7dbff Compare March 5, 2026 21:15
@cuviper
Copy link
Member

cuviper commented Mar 5, 2026

cc @Kobzol -- this is probably more like the bootstrap-oriented fix you had in mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants