-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Prepare beta 1.40.0 #66108
Prepare beta 1.40.0 #66108
Conversation
(rust_highfive has picked a reviewer for you, use r? to override) |
|
@bors r+ p=1000 |
📌 Commit 22f4fa7 has been approved by |
@bors rollup=never |
⌛ Testing commit 22f4fa7 with merge 648d04c697257e2e8579aef3cf8af3c4fc746ab5... |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
💔 Test failed - checks-azure |
@alexcrichton -- any thoughts on a good approach to resolving this failure? I suspect personally that we might be conflicting with the bootstrap bitflags, but not sure - I'm not sure why we're only seeing this in this PR (i.e., how master was working) |
I've got no idea why this is only on beta and not on master. This fixes it though: diff --git a/src/bootstrap/builder.rs b/src/bootstrap/builder.rs
index 2748903f2d4..de4e6b56459 100644
--- a/src/bootstrap/builder.rs
+++ b/src/bootstrap/builder.rs
@@ -886,6 +886,7 @@ impl<'a> Builder<'a> {
// things still build right, please do!
match mode {
Mode::Std => metadata.push_str("std"),
+ Mode::ToolRustc => metadata.push_str("tool-rustc"),
_ => {},
}
cargo.env("__CARGO_DEFAULT_LIB_METADATA", &metadata); I've got no idea why it fixes it though. |
It's super unclear why this broke when we switched to beta but not previously -- but at least it's hopefully fixed now.
@bors r+ Pushed up the fix suggested by @alexcrichton along with a comment that proposes one theory for why it's a fix -- though presumably an incomplete one, as it wouldn't explain different behavior between beta and master (hopefully). |
📌 Commit 73369f3 has been approved by |
Prepare beta 1.40.0 cc @Mark-Simulacrum
☀️ Test successful - checks-azure |
FWIW, wrt the rustdoc build fix: 1.38.0 can build 1.39.0 just fine, but 1.39.0 cannot build itself without backporting the patch. |
cc @Mark-Simulacrum