-
Notifications
You must be signed in to change notification settings - Fork 52
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
Bump to 1.6.0 #30
Bump to 1.6.0 #30
Conversation
It seems like installing / running miri is failing in CI, but when I do it locally, it works out fine. Any idea what's going on here @RalfJung? Should CI be doing |
This is odd, I have not seen that failure before. It does install the latest xargo, I am not sure what else there might be that could be causing trouble. Somehow the rustc-std-workspace-std patch does not seem to kick in? |
@alexcrichton something odd seems to happen, there's a version 1.0.1 of rustc-std-workspace-std on crates.io but the rustc workspace only contains a version 1.0.0? It looks like that version mismatch makes cargo pick the crates.io-version even though xargo set up a patch to use the local version instead. |
Hm, since you don't know the fix, I'm tempted to configure CI to allow this job to fail, with the hopes that this bug eventually gets fixed upstream :-/ Do you have a better plan than that? |
If my analysis is correct, the bug is outside of your or my control, so no, I do not have a better plan. |
Should be fixed in rust-lang/rust#64320 |
@alexcrichton that was quick, thanks! |
Hm ok so I retriggered these tests now that there are new nightlies out with Alex's fix, and now there are issues about
At this point, I'm going to mark the miri tests as "allow fail" until this stuff gets fixed upstream. |
The miri toolchain/infra is pretty flaky at the moment, unfortunately.
Yeah just saw these "backtrace" things on our CI as well. So far I have no idea what that is about. Really the biggest problem is that rust-lang/rust#60301 is still open, which means that we ship Miri even if we know the tests are broken. |
No description provided.