-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
ucx: 1.8.1 -> 1.9.0 #105377
ucx: 1.8.1 -> 1.9.0 #105377
Conversation
@GrahamcOfBorg test slurm |
Result of 19 packages marked as broken and skipped:
5 packages failed to build:
63 packages built:
|
Already broken on master:
source hash mismatch on master
Tests fail on master. Still please do not self merge without nixpkgs-review to make sure you are not introducing regressions. |
Thanks for double checking but I think in this particular case nixpkgs-review is not necessary. It is only a plugin of openmpi that depends on UCX. I think it sufficient here to check that openmpi works properly. Even though an update of UCX generates a lot of rebuilds it is unlikely that any other package is impacted. |
Motivation for this change
Regular upgrade:
https://github.com/openucx/ucx/blob/master/NEWS
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)