Skip to content

Prune dependency graph#165

Merged
anttiharju merged 7 commits intomainfrom
flake-surgery
Apr 6, 2026
Merged

Prune dependency graph#165
anttiharju merged 7 commits intomainfrom
flake-surgery

Conversation

@anttiharju
Copy link
Copy Markdown
Owner

Just try to minimise the number of packages that get installed. It's real nice that nix deduplicates a bunch of things

main method was

nix-store --query --tree $(nix build .#devShells.aarch64-darwin.default --print-out-paths --no-link) | sed 's|/nix/store/[a-z0-9]*-||g' | code -

and reasoning about it

Coreutils is here anyway, so no need to have the extra cruft
zig 0.16 not being available through an upstream nix distribution
is giving me pain
@anttiharju
Copy link
Copy Markdown
Owner Author

before:

$ docker images
REPOSITORY                              TAG       IMAGE ID       CREATED        SIZE
ghcr.io/anttiharju/compare-changes-ci   latest    fe05d2ac044e   56 years ago   4.11GB

@anttiharju
Copy link
Copy Markdown
Owner Author

after:

$ docker images
REPOSITORY                              TAG       IMAGE ID       CREATED        SIZE
ghcr.io/anttiharju/compare-changes-ci   current   bd10cefad333   56 years ago   4.13GB

lol it got bigger

@anttiharju
Copy link
Copy Markdown
Owner Author

well, relcheck still brings in the fat git and bumping the nur-packages ref is bit cumbersome due to the zig situation, I don't want to rebuild them because the cache setup is so finnicky and it takes a long time. And official release is soon anyway.

and once that is resolved zensical could be brought in. In the meanwhile I could rewrite find-changes-action in rust to get rid of that python dep

@anttiharju
Copy link
Copy Markdown
Owner Author

none of this makes a lot of sense in practice but I am just curious about all of this.

it is kind of magical to see the full fat dependency graph for my entire setup

@anttiharju
Copy link
Copy Markdown
Owner Author

and maybe I should submit a pr for action-validator packaging. Fix the version output with the patching and ensure gitMinimal is used as a new runtime dep.

@anttiharju
Copy link
Copy Markdown
Owner Author

the size growth is likely from the fact that we now have both git and gitMinimal. Most of the things removed were actually not that large anyway.

but zensical likely is a very good reduction in container image size because we can remove the fat python dep

@anttiharju anttiharju merged commit 498b62f into main Apr 6, 2026
4 checks passed
@anttiharju anttiharju deleted the flake-surgery branch April 6, 2026 17:10
anttiharju bot added a commit that referenced this pull request Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant