Skip to content

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Sep 9, 2024

Note sure if BOOTSTRAP_ARGS will make any difference here, but all the other x.py invocations have it and I did not deliberately leave it away when I added these, so... probably best to add them?

Also don't unnecessarily set BOOTSTRAP_SKIP_TARGET_SANITY while we are at it.

also don't unnecessarily set BOOTSTRAP_SKIP_TARGET_SANITY while we are at it
@rustbot
Copy link
Collaborator

rustbot commented Sep 9, 2024

r? @albertlarsan68

rustbot has assigned @albertlarsan68.
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

@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 Sep 9, 2024
Copy link
Member

@compiler-errors compiler-errors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The side-effect of this is to pass -v to these x.py miri invocations, so all it will be is more verbose. Seems fine to me.

@compiler-errors
Copy link
Member

r? compiler-errors @bors r+ rollup

@bors
Copy link
Collaborator

bors commented Sep 10, 2024

📌 Commit 62d196f has been approved by compiler-errors

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 10, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 10, 2024
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#130143 (miri-test-libstd: add missing BOOTSTRAP_ARGS)
 - rust-lang#130173 (rustdoc: add two regression tests)
 - rust-lang#130175 (`rustc_mir_transform` cleanups 3)
 - rust-lang#130184 (coverage: Clean up terminology in counter creation)
 - rust-lang#130185 (abi/compatibility test: remove tests inside repr(C) wrappers)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 10, 2024
…iaskrgr

Rollup of 5 pull requests

Successful merges:

 - rust-lang#130143 (miri-test-libstd: add missing BOOTSTRAP_ARGS)
 - rust-lang#130173 (rustdoc: add two regression tests)
 - rust-lang#130175 (`rustc_mir_transform` cleanups 3)
 - rust-lang#130184 (coverage: Clean up terminology in counter creation)
 - rust-lang#130185 (abi/compatibility test: remove tests inside repr(C) wrappers)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 34c935b into rust-lang:master Sep 10, 2024
6 checks passed
@rustbot rustbot added this to the 1.83.0 milestone Sep 10, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Sep 10, 2024
Rollup merge of rust-lang#130143 - RalfJung:miri-test-libstd, r=compiler-errors

miri-test-libstd: add missing BOOTSTRAP_ARGS

Note sure if BOOTSTRAP_ARGS will make any difference here, but all the other x.py invocations have it and I did not *deliberately* leave it away when I added these, so... probably best to add them?

Also don't unnecessarily set BOOTSTRAP_SKIP_TARGET_SANITY while we are at it.
@RalfJung RalfJung deleted the miri-test-libstd branch September 11, 2024 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

5 participants