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
hydra: 2018-08-07 -> 2019-02-01 #55101
Conversation
The change itself seems fine. I'll test it tonight against several Nix versions (and on my personal setup). Also, I'm wondering if we want to enhance the NixOS Hydra to test whether it works with all currently supported Nix versions (I can implement that tonight as well). |
The component `hydra-evaluator` requires `pool.hh` from Nix which isn't provided by Nix 1.x and thus fails with the following error in this case: ``` hydra-evaluator.cc:3:10: fatal error: pool.hh: No such file or directory #include "pool.hh" ^~~~~~~~~ compilation terminated. make[3]: *** [Makefile:443: hydra_evaluator-hydra-evaluator.o] Error 1 ``` As the commit is from 2016 and fairly hard to revert for Nix 1.x support, it's easier to drop that. This has been tested with fixed perl-bindings for Nix1 as done in NixOS#55146.
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.
The change itself seems fine. I pushed two fixes to this branch to drop the support for Nix1 and to build the Hydra test against multiple Nix versions. Just to be sure, I'll request a review from some more folks.
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.
I would prefer to create two separated tests, one with Nix stable and one with Nix unstable instead of having to boot two VMs at the same time. In case of failure, it will also be easier to know which version of Nix failed.
I updated the test accordingly: instead of a single test with multiple nodes and subtests, a dedicated testcase for each combination with Hydra and Nix is created. |
@nlewo would you mind having another look? |
Hydra should support multiple Nix versions (and currently contains fixes to work with Nix 2.0 and higher). Further Nix versions can be added to the `hydraPkgs` expression in the test case which lists all supported Nix versions for Hydra.
@GrahamcOfBorg test hydra.nixStable hydra.nixUnstable |
Motivation for this change
The version of Hydra currently in nixpkgs does not build because it is incompatible with the latest version of Nix.
Things done
Updates Hydra to the latest revision.
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)