Skip to content

Commit

Permalink
libarchive: make static patch unconditional
Browse files Browse the repository at this point in the history
Cabal 3.8 has the same requirements as pkgsStatic even when linking
dynamically, so this patch will be useful for compiling
haskellPackages.libarchive.

haskell/cabal#8455
  • Loading branch information
sternenseemann authored and fabaff committed Aug 26, 2023
1 parent f2c9fe4 commit 4fc3946
Showing 1 changed file with 12 additions and 14 deletions.
26 changes: 12 additions & 14 deletions pkgs/development/libraries/libarchive/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ let
autoreconfHook = buildPackages.autoreconfHook269;
in
assert xarSupport -> libxml2 != null;
(stdenv.mkDerivation (finalAttrs: {
stdenv.mkDerivation (finalAttrs: {
pname = "libarchive";
version = "3.6.2";

Expand All @@ -43,6 +43,16 @@ assert xarSupport -> libxml2 != null;

outputs = [ "out" "lib" "dev" ];

patches = [
# fixes static linking; upstream in releases after 3.6.2
# https://github.com/libarchive/libarchive/pull/1825 merged upstream
(assert finalAttrs.version == "3.6.2"; fetchpatch {
name = "001-only-add-iconv-to-pc-file-if-needed.patch";
url = "https://github.com/libarchive/libarchive/commit/1f35c466aaa9444335a1b854b0b7223b0d2346c2.patch";
hash = "sha256-lb+zwWSH6/MLUIROvu9I/hUjSbb2jOWO755WC/r+lbY=";
})
];

postPatch = let
skipTestPaths = [
# test won't work in nix sandbox
Expand Down Expand Up @@ -119,16 +129,4 @@ assert xarSupport -> libxml2 != null;
passthru.tests = {
inherit cmake nix samba;
};
})).overrideAttrs(previousAttrs:
assert previousAttrs.version == "3.6.2";
lib.optionalAttrs stdenv.hostPlatform.isStatic {
patches = [
# fixes static linking; upstream in releases after 3.6.2
# https://github.com/libarchive/libarchive/pull/1825 merged upstream
(fetchpatch {
name = "001-only-add-iconv-to-pc-file-if-needed.patch";
url = "https://github.com/libarchive/libarchive/commit/1f35c466aaa9444335a1b854b0b7223b0d2346c2.patch";
hash = "sha256-lb+zwWSH6/MLUIROvu9I/hUjSbb2jOWO755WC/r+lbY=";
})
];
})
})

0 comments on commit 4fc3946

Please sign in to comment.