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
nim: 0.18.0 -> 0.19.0 #49250
nim: 0.18.0 -> 0.19.0 #49250
Conversation
- remove sqlite as dependency (I don't see why it's needed) - run checkPhase after installPhase (at least one test assumes the standard library in ../lib relative to the nim binary) - the broken tests pass now or don't exist anymore - two of the tests requiring network access pass now without network access - the tests in manyloc do not download dependencies and are passing now
@GrahamcOfBorg build nim |
Failure on aarch64-linux (full log) Attempted: nim Partial log (click to expand)
|
Timed out, unknown build status on x86_64-darwin (full log) Attempted: nim Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: nim Partial log (click to expand)
|
I tried to fix the failure on Aarch64.
|
If ofborg timeouts this is ok. Our hydra has longer timeouts. |
@GrahamcOfBorg build nim |
Success on aarch64-linux (full log) Attempted: nim Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: nim Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: nim Partial log (click to expand)
|
@@ -30,10 +29,16 @@ stdenv.mkDerivation rec { | |||
|
|||
buildInputs = [ | |||
makeWrapper nodejs-slim-8_x tzdata coreutils |
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.
makeWrapper tzdata coreutils
should be in nativeBuildInputs
(they're needed only in installPhase and checkPhase)
Build + tests successful on nixos-18.09 otherwise |
thanks for reviewing. I moved the packages you mentioned to Also I updated |
@royneary You need to add the @GrahamcOfBorg build nrpl ( |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: nrpl Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: nrpl Partial log (click to expand)
|
@gebner fixed. |
|
standard library in ../lib relative to the nim binary)
Motivation for this change
Update nim to 0.19.0.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)