• We probably want to disable shutdown ramfs in things like containers
• We probably don't want to strip at all for shutdown ramfs
• We should probably use targetPrefix to get the right strip from binutils for building initrd

In reply to @elvishjerricco:matrix.org
I want to test it with compression just in case

In reply to @janne.hess:helsinki-systems.de
I did it this way because symbols should have a better compression ratio imo and that would affect test results

In reply to @linus:schreibt.jetzt
IMHO we should continue stripping in the drv but drop it for shutdown

In reply to @elvishjerricco:matrix.org
That still requires builders to have binutils, which isn't necessarily the case without stripping :/ Maybe make it a config option?

In reply to @linus:schreibt.jetzt

oh that's a lot more extensive than what I'm currently building:

diff --git a/pkgs/build-support/kernel/make-initrd-ng/src/main.rs b/pkgs/build-support/kernel/make-initrd-ng/src/main.rs
index 294c570a374..681fc5468fd 100644
--- a/pkgs/build-support/kernel/make-initrd-ng/src/main.rs
+++ b/pkgs/build-support/kernel/make-initrd-ng/src/main.rs
@@ -57,12 +57,11 @@ fn copy_file<P: AsRef<Path> + AsRef<OsStr>, S: AsRef<Path> + AsRef<OsStr>>(
 ) -> Result<(), Error> {
     fs::copy(&source, &target)?;

-    if !Command::new("ldd").arg(&source).output()?.status.success() {
-        // Not dynamically linked - no need to recurse
-        return Ok(());
-    }
-
-    let rpath_string = patch_elf("--print-rpath", &source)?;
+    let rpath_string = match patch_elf("--print-rpath", &source) {
+        // Not dynamically linked probably
+        Err(_) => { return Ok(()); },
+        Ok(s) => s,
+    };
     let needed_string = patch_elf("--print-needed", &source)?;
     // Shared libraries don't have an interpreter
     if let Ok(interpreter_string) = patch_elf("--print-interpreter", &source) {