-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
cargo,clippy,rustc,rustfmt: 1.77.2 -> 1.78.0 #309580
Conversation
Some changes were made when this patch was committed to rust-next. Most importantly, the minimum rustc version was updated from 1.77.0 to 1.77.1, and if we use the latest version of the patch, we'll be able to cleanly apply the 1.78.0 patch. rust-next gets force pushed sometimes[1], so we shouldn't fetch from it in a FOD, hence we now have rust-1.77-6.8.patch in-tree, but this will save us from having rust-1.78.patch in-tree, which we can fetch from lore. [1]: https://github.com/Rust-for-Linux/linux/activity?ref=rust-next
Built. |
Is this ready to go? |
Yes from my PoV, but since we're between staging cycles anyway, I was holding off on merging in case anybody wanted to review. |
I'm not 100% sure how git auto-resolved |
sigh the update seems to have broken cross compilation of Rust programs. Bisecting… |
|
Fix for cross compilation: #317179 |
Potential fix for aarch64-darwin: #317273 |
Description of changes
I guess we want to hold this until breaking changes are allowed again, since we have packages using nightly features that have been renamed.
Built:
I haven't fixed thunderbirdPackages.thunderbird-102, but it's been marked insecure since September, and I think we should just remove it: #308820.
Could somebody please check fd on x86_64-darwin?
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.