Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

when Miri tests are not passing, do not add Miri component #66053

Merged
merged 4 commits into from
Nov 5, 2019

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented Nov 3, 2019

Second attempt, this time based on the JSON files that exist since #65274.

Fixes #60301
r? @pietroalbini @alexcrichton

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 3, 2019
src/tools/build-manifest/src/main.rs Outdated Show resolved Hide resolved
src/tools/build-manifest/src/main.rs Show resolved Hide resolved
@pietroalbini
Copy link
Member

@bors r+

Thanks! If Alex has comments feel free to r-.

@bors
Copy link
Contributor

bors commented Nov 4, 2019

📌 Commit 224378c has been approved by pietroalbini

@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 Nov 4, 2019
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Nov 5, 2019
…bini

when Miri tests are not passing, do not add Miri component

Second attempt, this time based on the JSON files that exist since rust-lang#65274.

Fixes rust-lang#60301
r? @pietroalbini @alexcrichton
bors added a commit that referenced this pull request Nov 5, 2019
Rollup of 8 pull requests

Successful merges:

 - #65948 (Improve MaybeUninit::get_{ref,mut} documentation)
 - #65953 (Allow specifying LLVM's MCTargetOptions::ABIName in target specification files)
 - #66012 (De-querify `trivial_dropck_outlives`.)
 - #66025 (`Span` cannot represent `span.hi < span.lo`)
 - #66047 (Don't double-count `simd_shuffle` promotion candidates)
 - #66053 (when Miri tests are not passing, do not add Miri component)
 - #66082 (clean highlightSourceLines code)
 - #66091 (Implemented the home_dir for VxWorks)

Failed merges:

r? @ghost
@bors bors merged commit 224378c into rust-lang:master Nov 5, 2019
@RalfJung RalfJung deleted the miri-toolstate branch November 6, 2019 10:05
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Feb 12, 2021
…ark-Simulacrum

Don't release Miri if its tests only failed on Windows

Extends rust-lang#66053 to Windows, so the released Miri won't be broken if its tests only fail on Windows.

Relevant Zulip thread: https://rust-lang.zulipchat.com/#narrow/stream/269128-miri/topic/Miri.20is.20still.20available.20in.20rustup.20today.3F
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 13, 2021
…k-Simulacrum

Don't release Miri if its tests only failed on Windows

Extends rust-lang#66053 to Windows, so the released Miri won't be broken if its tests only fail on Windows.

Relevant Zulip thread: https://rust-lang.zulipchat.com/#narrow/stream/269128-miri/topic/Miri.20is.20still.20available.20in.20rustup.20today.3F
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

miri component gets shipped even though the tests fail
5 participants