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
niv: reduce the closure size when using the executable #80644
Conversation
As the most intended use for the tool is in the CLI, we should only get the executable with a small closure size.
@GrahamcOfBorg build niv |
I don't see any improvement when following this PR. The size is the same.
|
@knl could it be that you were downloading all build dependencies, so that the executable could be built by nix on your machine? It doesn't appear to be present in cache.nixos.org. Note: I've updated the installation instructions for using niv's cache. |
I can testify that these are the dependencies I had downloaded when using the command at the installation instructions:
Most of which don't seem like build dependencies to me.. |
Mh, yeah. They're needed to build |
So this is an upstream issue for Niv and not for Nixpkgs? Shall we close
thise then?
|
The issue brought up by @doronbehar is definitely upstream, I'll have to fix it there. Re. this particular issue, I can't reproduce. I think we can close when @knl confirms #80644 (comment) |
FTR @doronbehar the niv issue is tracked in nmattia/niv#204 and should be fixed in nmattia/niv#205 |
@doronbehar - @nmattia's fix in nmattia/niv#205 indeed makes the downloaded size smaller. Closing this PR as it should have started as a Draft. |
Motivation for this change
As the most intended use for the tool is in the CLI, we should only get
the executable with a small closure size.
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)