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
rustscan: 1.8.0 -> 1.10.1 #98410
rustscan: 1.8.0 -> 1.10.1 #98410
Conversation
Fails to build on NixOS. This seems to be because one of the tests got split. I am currently testing this change:
|
With the above changes, the package builds and runs fine on NixOS. |
aa713b7
to
4feca01
Compare
Changed. I built it locally and nothing failed. Are the tests more strict on nixOS compared to nix? And why is graham green if the tests fail in nixOS? I am a bit confused right now. Can you maybe clear this up or reference some documentation about this? |
Perhaps the sandbox is off by default on foreign distributions? Also, I had the impression that the CI hooked up to GitHub PRs only checks whether the derivations instantiate without errors... |
Sandbox is disabled on Debian and Ubuntu because it sometimes causes really strange errors when building certain packages. Not ideal for package maintenance but I try to keep it in mind and maybe change that in the future. |
1779b04
to
c29f88d
Compare
@NieDzejkob Could you maybe take another look if it still builds/passes tests on nixOS? (I should setup some VM...) |
On Tue Sep 29, 2020 at 2:16 PM CEST, Sandro wrote:
@NieDzejkob Could you maybe take another look if it still builds/passes
tests on nixOS? (I should setup some VM...)
you could also try enabling the sandbox in your configuration file on
your machine. That should catch most issues ;)
I'll take a look at it later.
|
Then some nodePackages are not building. |
|
9ba36c8
to
ef52d44
Compare
I finally set up a Vagrant box to properly test with sandbox. 🎉 |
ef52d44
to
422a52c
Compare
Motivation for this change
Keep things up to date.
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)