-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Make rustc_parse_format
compile on stable again
#117819
Conversation
// WARNING: We want to be able to build this crate with a stable compiler, | ||
// so no `#![feature]` attributes should be added! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Someday I might open an MCP for #![deny(rustc::features)]
or a lang RFC for #![deny(features)]
:^)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think a simple PR to add the lint would be enough for that. Lints are explicitly not stable, and this is a lint for an unstable feature (the feature attribute).
can we add a CI builder that builds those crates with the bootstrap compiler (without RUSTC_BOOTSTRAP)? |
That's not my metier but I can read up on it. |
Can we merge this as is? |
@bors r+ feel free to r- and r=me with an even bigger and scarier warning if you want to |
cc #117937 |
☀️ Test successful - checks-actions |
Finished benchmarking commit (d4559c0): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
CyclesResultsThis is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.
Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 674.821s -> 674.873s (0.01%) |
Fixes #115948 (review comment).
cc @Veykril @notriddle
r? compiler