-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
dhcpcd: 10.0.6 -> 10.1.0 #284123
dhcpcd: 10.0.6 -> 10.1.0 #284123
Conversation
Trick question -- why a |
@deepfire greetings ! can you check that link to the issue you have posted is correct ?
silly answer would be because it has a lot of direct and indirect reverse dependencies, are you after any specific dependency ? EDIT: networkmanager depends on dhcpcd, which means pretty much all desktop environments need to be rebuilt |
@kirillrdy, my apologies, I've used a repo-local link -- here's the proper link to the issue: NetworkConfiguration/dhcpcd#291 |
I guess that's the essence of my actual question -- I've seen that the full path to |
not sure what those are. but for as long as anything uses store path its unavoidable. and using store paths is the whole point of nix, so that this precise version of NetworkManager was built/tested with that precise version of dhcpcd |
Sorry, I garbled the overlong name 0f NetworkManager.
|
maybe in the future https://nixos.wiki/wiki/Ca-derivations can fix this |
Updated, as Nixpkgs |
cc @kirillrdy |
Maybe this should target staging because of the mass rebuild? |
This works around NixOS#291, halfway.
Rebases suggested by the approving reviewer are disincentivized, I see ; -) |
This works around NetworkConfiguration/dhcpcd#291, by bumping
dhcpcd
to the latest released.Description of changes
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.