Merge pull request #173890 from risicle/ris-flann-clang-fix

flann: don't unbundle lz4 for clang
main
Ben Siraphob 2 years ago committed by GitHub
commit bf6b5141b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      pkgs/development/libraries/flann/default.nix

@ -32,6 +32,7 @@ stdenv.mkDerivation rec {
url = "https://salsa.debian.org/science-team/flann/-/raw/debian/1.9.1+dfsg-9/debian/patches/0001-src-cpp-fix-cmake-3.11-build.patch";
sha256 = "REsBnbe6vlrZ+iCcw43kR5wy2o6q10RM73xjW5kBsr4=";
})
] ++ lib.optionals (!stdenv.cc.isClang) [
# Avoid the bundled version of LZ4 and instead use the system one.
(fetchpatch {
url = "https://salsa.debian.org/science-team/flann/-/raw/debian/1.9.1+dfsg-9/debian/patches/0003-Use-system-version-of-liblz4.patch";
@ -57,7 +58,8 @@ stdenv.mkDerivation rec {
unzip
];
propagatedBuildInputs = [ lz4 ];
# lz4 unbundling broken for llvm, use internal version
propagatedBuildInputs = lib.optional (!stdenv.cc.isClang) lz4;
buildInputs = lib.optionals enablePython [ python3 ];

Loading…
Cancel
Save