-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Fix rustdoc scrape examples crash #145008
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
base: master
Are you sure you want to change the base?
Fix rustdoc scrape examples crash #145008
Conversation
|
This comment has been minimized.
This comment has been minimized.
@@ -0,0 +1,7 @@ | |||
pub struct X; |
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.
Btw, tester.rs
must exist, without it, it doesn't crash.
This comment has been minimized.
This comment has been minimized.
d29e4f3
to
ff371fe
Compare
This comment has been minimized.
This comment has been minimized.
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.
Since I also don't have the time to investigate the root cause, a hotfix seems fine to me.
Regarding the test, I'd rather see a test that doesn't rely on Cargo. Let me see if I can cook sth. up. But otherwise r=me if I don't reply within the next hours.
Eh, r=me once CI green |
ff371fe
to
a2b1714
Compare
|
Thank you! We really appreciate it. |
@bors r=fmease rollup |
…examples-crash, r=fmease Fix rustdoc scrape examples crash Fixes rust-lang#144752. The regression was introduced in rust-lang#144600. Although I don't understand why it is an issue currently, this allows to bypass the failure for now until we can figure out what's wrong as it's currently blocking new `bevy`'s release. cc `@alice-i-cecile` r? `@fmease`
Rollup of 15 pull requests Successful merges: - #144195 (Parser: Recover from attributes applied to types and generic args) - #144794 (Port `#[coroutine]` to the new attribute system) - #144835 (Anonymize binders in tail call sig) - #144861 (Stabilize `panic_payload_as_str` feature) - #144917 (Enforce tail call type is related to body return type in borrowck) - #144948 (we only merge candidates for trait and normalizes-to goals) - #144956 (Gate const trait syntax) - #144970 (rustdoc: fix caching of intra-doc links on reexports) - #144972 (add code example showing that file_prefix treats dotfiles as the name of a file, not an extension) - #144975 (`File::set_times`: Update documentation and example to support setting timestamps on directories) - #144977 (Fortify generic param default checks) - #144996 (simplifycfg: Mark as changed when start is modified in collapse goto chain) - #144998 (mir: Do not modify NonUse in `super_projection_elem`) - #145000 (Remove unneeded `stage` parameter when setting up stdlib Cargo) - #145008 (Fix rustdoc scrape examples crash) r? `@ghost` `@rustbot` modify labels: rollup
Fixes #144752.
The regression was introduced in #144600. Although I don't understand why it is an issue currently, this allows to bypass the failure for now until we can figure out what's wrong as it's currently blocking new
bevy
's release.cc @alice-i-cecile
r? @fmease