Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

use util-linuxMinimal instead of util-linux for column #472

Merged
merged 1 commit into from
Mar 9, 2023

Conversation

bobvanderlinden
Copy link
Contributor

Currently the util-linux package is used for the column cli. This is the only dependency for which util-linux is used. It results in quite a significant addition to the closure size.

util-linuxMinimal is somewhat smaller and also includes the column command.

I used nix-tree to check the closure size of the devenv command:

Before:

devenv

After:
devenv2

It is still big, but at least it is smaller 馃槄

Since python is already part of the closure, it might be a good idea to base everything on python and remove dependencies on util-linux-minimal, findutils, coreutils and jq.

This could help CI builds quite a bit in reducing the time it takes to fetch devenv.sh.

Currently the util-linux package is used for the column cli. This is the
only dependency for which util-linux is used. It results in quite a
significant addition to the closure size.

util-linuxMinimal is somewhat smaller and also includes the column
command.
@domenkozar domenkozar merged commit 4fd2e77 into cachix:main Mar 9, 2023
@bobvanderlinden bobvanderlinden deleted the pr-decrease-closure-size branch March 9, 2023 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants