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
buildGoPackage: enable strictDeps #82786
Conversation
I will move this over to staging once I fixed some stuff. |
f32cc2c
to
da97f74
Compare
4333691
to
d04dbd7
Compare
This will improve cross compiling in the long run. See also NixOS#82786
83b578b
to
7b03b99
Compare
(N.B. In the other, I say go with |
I may not have yet understand very well the purpose of Why in the case of |
@romildo There is some documentation, though not enough, in https://nixos.org/nixpkgs/manual/#chap-cross . Basically, By not having native be more permissive than cross, we make the cross builds much less likely to bitrot. |
7b03b99
to
4f2d52c
Compare
pkgs/applications/version-management/git-and-tools/grv/default.nix
Outdated
Show resolved
Hide resolved
@@ -31,7 +31,21 @@ in stdenv.mkDerivation { | |||
avrgcc | |||
avrbinutils | |||
gcc-arm-embedded | |||
gcc-armhf-embedded |
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.
7cc0f84
to
dcb75b8
Compare
In order to improve cross-compilation let's enable strictDeps and fix what breaks.
dcb75b8
to
b4d144e
Compare
@GrahamcOfBorg build dnscrypt-proxy2 |
Cherry-picked to ff2ea91 in staging |
Motivation for this change
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)