Skip to content

Commit

Permalink
Fix buildNoTest
Browse files Browse the repository at this point in the history
`checkInputs` is not right for this because we don't just need these
deps when `doTest`, we also need them when `installUnitTests`.
  • Loading branch information
Ericson2314 committed Jan 2, 2024
1 parent 2b20f36 commit 86e9244
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,9 @@ in {
] ++ lib.optionals (!stdenv.hostPlatform.isWindows) [
editline
lowdown
] ++ lib.optionals buildUnitTests [
gtest
rapidcheck
] ++ lib.optional stdenv.isLinux libseccomp
++ lib.optional stdenv.hostPlatform.isx86_64 libcpuid
# There have been issues building these dependencies
Expand All @@ -232,11 +235,6 @@ in {
dontBuild = !attrs.doBuild;
doCheck = attrs.doCheck;

checkInputs = [
gtest
rapidcheck
];

nativeCheckInputs = [
git
mercurial
Expand Down

0 comments on commit 86e9244

Please sign in to comment.