Skip to content

Commit

Permalink
gzip: use makeShellWrapper instead of makeWrapper
Browse files Browse the repository at this point in the history
makeWrapper is in default makeShellWrapper but sometimes it is
beneficial to switch it in overlay to makeBinaryWrapper, but gzip is
injecting shell expansion that doesn't work binary wrapper. Thus this
takes shell wrapper explicitly as a dependency now.
  • Loading branch information
Cynerd committed Oct 25, 2024
1 parent e9eca9b commit 5dd6467
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkgs/tools/compression/gzip/default.nix
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{ lib, stdenv
, fetchurl
, makeWrapper
, makeShellWrapper
, updateAutotoolsGnuConfigScriptsHook
, runtimeShellPackage
}:
Expand All @@ -23,7 +23,7 @@ stdenv.mkDerivation rec {

enableParallelBuilding = true;

nativeBuildInputs = [ updateAutotoolsGnuConfigScriptsHook makeWrapper ];
nativeBuildInputs = [ updateAutotoolsGnuConfigScriptsHook makeShellWrapper ];
buildInputs = [ runtimeShellPackage ];

makeFlags = [
Expand Down

0 comments on commit 5dd6467

Please sign in to comment.