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
ISO/OVA closure size reductions #47296
Conversation
This is already done in installer/cd-dvd/installation-cd-graphical-kde.nix but not in profiles/graphical.nix. Related to NixOS#47256.
nfs-utils had a dependency on gcc through etc/systemd/system-generators/*-server-generator. It was not stripped correctly because it’s not in an expected path. This adds it to the strip list.
6854808
to
1427c50
Compare
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: nfs-utils Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: nfs-utils Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: nfs-utils Partial log (click to expand)
|
@GrahamcOfBorg test nfs3 |
Success on x86_64-linux (full log) Attempted: tests.nfs3 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tests.nfs3 Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: tests.nfs4 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: tests.nfs4 Partial log (click to expand)
|
Is this okay to merge into master? 101-500 shouldn't be too much... |
@matthewbauer should we backport this to 18.09 ? |
Yeah, master looks fine. |
@@ -83,6 +83,8 @@ in stdenv.mkDerivation rec { | |||
"statdpath=$(TMPDIR)" | |||
]; | |||
|
|||
stripDebugList = [ "lib" "libexec" "bin" "etc/systemd/system-generators" ]; | |||
|
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.
Compiled binaries in etc/**
😱
According to Hydra the graphical ISO has exactly the same size and OVA won't evaluate.
|
The OVA eval error is unrelated, it was caused by #34409 which was reverted a few hours ago. |
Right. Hydra shows the OVA size did go down by roughly 25 MiB. |
Motivation for this change
A few things to partially address #47292. Gets rid of GCC and Qt4 from the .ova build products (and GCC from the .iso)