Conversation
|
|
|
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
It is always my preference to have changes in a release but I am happy to acknowledge it is merely a preference. |
|
I think we need to explicitly include a dependency on https://crates.io/crates/windows-link on the appropriate platform. |
|
A release would be nice to unblock #145516 |
|
I created a PR to fix this #151680. The problem is backtrace and std need to be updated in tandem due to changes in how we use windows-bindgen. |
Update backtrace and windows-bindgen Supersedes the backtrace bump in rust-lang#151659 This is mostly just renaming `windows_targets` to `windows_link` but it needs to be done in tandem with the backtrace submodule update. The reason for doing this is that backtrace is both copy/pasted into std (via being a submodule) and published as an independent crate.
Rollup merge of #151680 - ChrisDenton:bindgen, r=tgross35 Update backtrace and windows-bindgen Supersedes the backtrace bump in #151659 This is mostly just renaming `windows_targets` to `windows_link` but it needs to be done in tandem with the backtrace submodule update. The reason for doing this is that backtrace is both copy/pasted into std (via being a submodule) and published as an independent crate.
Update backtrace and windows-bindgen Supersedes the backtrace bump in rust-lang/rust#151659 This is mostly just renaming `windows_targets` to `windows_link` but it needs to be done in tandem with the backtrace submodule update. The reason for doing this is that backtrace is both copy/pasted into std (via being a submodule) and published as an independent crate.
…t, r=tgross35 Bump `std`'s `backtrace`'s `rustc-demangle` Alternative to rust-lang#151659. Fixes rust-lang#151548.
…ss35 Bump `std`'s `backtrace`'s `rustc-demangle` Alternative to rust-lang/rust#151659. Fixes rust-lang/rust#151548.
…ss35 Bump `std`'s `backtrace`'s `rustc-demangle` Alternative to rust-lang/rust#151659. Fixes rust-lang/rust#151548.
Updates git submodule
backtracemainly to include the changes from PR rust-lang/backtrace-rs#746. Note that these changes haven't been released yet, although that shouldn't really matter or do you see it as a requirement?Locally I've confirmed that it fixes #151548. I don't think it warrants a regression test (esp. since backtrace tests are rather fragile aren't they?).
I'm actually not sure if only bumping the version of
rustc-demangleinlibrary/std/Cargo.tomlforbacktracewould've sufficed 😅. I went to great lengths updatingbacktracesince I had to face "package version conflicts" and similar back in PR #150843 when I pointedrustc-demangleandbacktraceto my personal forks. In hindsight though I guess these errors occurred for other reasons.r? workingjubilee (rust-lang/backtrace-rs#746) or reassign