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
update most packages I maintain #72020
Conversation
let me know when you're done adding packages, and then I'll review :) |
@jonringer Done 😄 |
@jonringer All is done. Thanks for suggestions! |
Became incorrect after #66585
next time, I would just group these into related packages, and do a PR per group. Harder to review when a lot is going on :) |
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.
nix-review
passes on NixOS
diff LGTM
no regressions
most packages with executables at least display --help
[29 built, 173 copied (912.2 MiB), 211.8 MiB DL]
https://github.com/NixOS/nixpkgs/pull/72020
18 package were build:
gnome3.gnome-boxes kmsxx openjk openmw-tes3mp openxray pakcs phodav python27Packages.kmsxx python37Packages.kmsxx python38Packages.kmsxx remmina spice-gtk video2midi virt-manager virt-manager-qt virt-viewer vk-messenger vkquake
@jonringer Thank you! |
Got it, ok 😄 |
Motivation for this change
Test with
nix-review pr 72020
Few packages were already bumped by other people
openmpt123 #71080
franz #71862
ps_mem #72611
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @