Skip to content

miri no longer builds after rust-lang/rust#61625 #61662

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
rust-highfive opened this issue Jun 8, 2019 · 1 comment · Fixed by #61743
Closed

miri no longer builds after rust-lang/rust#61625 #61662

rust-highfive opened this issue Jun 8, 2019 · 1 comment · Fixed by #61743
Assignees
@eddyb @RalfJung @oli-obk
Labels
T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@rust-highfive
Copy link
Contributor

Hello, this is your friendly neighborhood mergebot.
After merging PR #61625, I observed that the tool miri no longer builds.
A follow-up PR to the repository https://github.com/rust-lang/miri is needed to fix the fallout.

cc @RalfJung, do you think you would have time to do the follow-up work?
If so, that would be great!

cc @oli-obk, the PR reviewer, and @rust-lang/compiler -- nominating for prioritization.
@rust-highfive rust-highfive added I-nominated T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 8, 2019
@RalfJung
Copy link
Member

Miri is fixed upstream, but will be broken again by #61673 so I'd like to land that before updating.

@RalfJung RalfJung mentioned this issue Jun 11, 2019
Merged
bors added a commit that referenced this issue Jun 13, 2019
@bors
Auto merge of #61743 - RalfJung:miri, r=oli-obk
96636f3 
update miri

Fixes #61662

r? @oli-obk
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@eddyb eddyb

@RalfJung RalfJung

@oli-obk oli-obk

Labels
T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update miri RalfJung/rust
4 participants
@eddyb @RalfJung @oli-obk @rust-highfive