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: Dont copy sources to $out #43328
Conversation
I believe it was meant for Go packages to depend on each other, but this was never realized. I think it is good to remove sources from This should be checked with |
@orivej As I suspected then. I'm running |
|
Some sources are quite large and they don't seem to offer much value
buildGoPackage no longer copy sources to $out
123828a
to
eb31dc6
Compare
Failure on aarch64-linux (full log) Attempted: go-ethereum Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: go-ethereum Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: go-ethereum Partial log (click to expand)
|
Motivation for this change
I don't see the reason to copy sources that are potentially quite large, especially with the go community practices of vendoring dependencies.
Do these sources serve any purpose?
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)CCing maintainers & others who have touched this recently @pbogdan @Mic92 @yegortimoshenko @kamilchm @lethalman @cstrahan @orivej @velovix