From c8052a3143bddc7917c28598406fc169fad4e4e7 Mon Sep 17 00:00:00 2001 From: Eli <88557639+lishaduck@users.noreply.github.com> Date: Mon, 3 Jun 2024 12:54:41 -0500 Subject: [PATCH] oils-for-unix: reenable binary stripping It seems that pkg.oils-for-unix was just copied from pkg.oil, so it didn't reenable stripping the binary. As the comment says, the old oil.ovm file had an appended zip file. The new C++ interpreter doesn't, so this should be safe. --- pkgs/by-name/oi/oils-for-unix/package.nix | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkgs/by-name/oi/oils-for-unix/package.nix b/pkgs/by-name/oi/oils-for-unix/package.nix index f477d66d8a07..316ba4a15030 100644 --- a/pkgs/by-name/oi/oils-for-unix/package.nix +++ b/pkgs/by-name/oi/oils-for-unix/package.nix @@ -51,9 +51,6 @@ stdenv.mkDerivation rec { "--readline=${readline-all}" ]; - # Stripping breaks the bundles by removing the zip file from the end. - dontStrip = true; - meta = { description = "A Unix shell with JSON-compatible structured data. It's our upgrade path from bash to a better language and runtime."; homepage = "https://www.oilshell.org/";