-
Notifications
You must be signed in to change notification settings - Fork 13.5k
Stage0 step #66156
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
Stage0 step #66156
Conversation
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 |
☔ The latest upstream changes (presumably #66175) made this pull request unmergeable. Please resolve the merge conflicts. |
9908679
to
b1b9ce7
Compare
Rebased. Somehow on_unimplemented de-stabilized? Not sure. |
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 |
@Centril I can't seem to get on_unimplemented working -- if I remove it, then I get:
but if I leave it in, then I get:
|
@Mark-Simulacrum Odd; What I did in #65794 was to move the attribute under |
libstd does have the rustc_attrs attribute. I wonder if maybe I should keep the cfg(bootstrap) around... |
Sure, if that makes things compile, why not. We can remove it next time. :) |
b1b9ce7
to
f4de461
Compare
@pietroalbini Okay, looks like this is compiling successfully now. |
Sorry for the delay reviewing this! @bors r+ rollup=never |
📌 Commit f4de4618207160f8e421e0a6d25c3ce9cbd03415 has been approved by |
@bors p=1 We should try to get nightly version bumped as soon as possible. |
⌛ Testing commit f4de4618207160f8e421e0a6d25c3ce9cbd03415 with merge b7dc243ca82da771aa8ef10e43682395bd9d792e... |
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 |
It's super unclear why this broke when we switched to beta but not previously -- but at least it's hopefully fixed now.
f4de461
to
994d836
Compare
@bors r=pietroalbini |
📌 Commit 994d836 has been approved by |
☀️ Test successful - checks-azure |
r? @pietroalbini