-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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
nixUnstable: 2.4pre20201102_550e11f -> 2.4pre20201105_387f824 #102932
Conversation
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.
While the change itself seems fine, I'd like to mention that Hydra doesn't build stuff anymore in hydra-queue-runner
which can be reproduced by running nixos/tests/hydra
on master
and on this branch.
Unfortunately I don't have the time right now to look at this, but since Hydra requires nixUnstable
these days, this is something that should be taken care of (cc @edolstra).
After a brief check I identified NixOS/nix@c4d903d and NixOS/nix@b43c13a to be the cause, however I acknowledge that those are relevant fixes and I don't know enough about that to patch Hydra accordingly.
So just realized that with a nix-daemon on latest nix master (i.e. this revision) all started builds stall, even without Hydra making a rollback to a previous configuration necessary. Hence 👎 on merging this. |
I agree with the statement from @Ma27 |
06815e5
to
5234865
Compare
NixOS/nix#4235 is currently the only remaining issue. I've added a patch for it to make this branch useful for testing. |
4e579b5
to
5fcf9c5
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/to-flake-or-not-to-flake/10047/5 |
Closing in favor of #104289, in order not to break any pins on this old branch. |
Motivation for this change
Most notably
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)