From 523280788e13df2bd72774e29db31b8fc068d9ec Mon Sep 17 00:00:00 2001
From: John Parent <john.parent@kitware.com>
Date: Thu, 10 Oct 2024 14:46:40 -0400
Subject: [PATCH 1/2] blosc/CMakeLists.txt: Update Lz4 handling

Lz4 is capable of vendoring its own CMake config
module, which is considerd preferable over vendored
find modules according to CMake best practices.

This patch adds support for importing an LZ4 install
detected via its CMake Config module by searching
for the targets imported via that module.
It then updates the link interface for cblosc
static and shared variants to link to their
respective LZ4 libraries (or whichevere is available)
---
 blosc/CMakeLists.txt | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/blosc/CMakeLists.txt b/blosc/CMakeLists.txt
index 7e6b765c..55a40ae0 100644
--- a/blosc/CMakeLists.txt
+++ b/blosc/CMakeLists.txt
@@ -88,7 +88,32 @@ endif(WIN32)
 
 if(NOT DEACTIVATE_LZ4)
     if(LZ4_FOUND)
-        set(LIBS ${LIBS} ${LZ4_LIBRARY})
+        # Check if LZ4 was detected from a conig
+        # module first
+        if(TARGET LZ4::lz4_shared)
+            set(SHARED_LIBS LZ4::lz4_shared)
+        endif()
+        if(TARGET LZ4::lz4_static)
+            set(STATIC_LIBS LZ4::lz4_static)
+        endif()
+        if(SHARED_LIBS AND STATIC_LIBS)
+            # Set a genex to match the appropriate library type respective to the
+            # type of the LZ4 library
+            set(LIBS
+                ${LIBS}
+                $<$<STREQUAL:$<TARGET_PROPERTY:Foo,TYPE>,SHARED_LIBRARY>:LZ4::lz4_shared>
+                $<$<NOT:$<STREQUAL:$<TARGET_PROPERTY:Foo,TYPE>,SHARED_LIBRARY>>:LZ4::lz4_static>
+            )
+        elseif(SHARED_LIBS OR STATIC_LIBS)
+            if(SHARED_LIBS)
+                set(LIBS ${LIBS} ${SHARED_LIBS})
+            else()
+                set(LIBS ${LIBS} ${STATIC_LIBS})
+            endif()
+        elseif(NOT SHARED_LIBS AND NOT STATIC_LIBS)
+            # Fallback to cblosc vendored find module
+            set(LIBS ${LIBS} ${LZ4_LIBRARY})
+        endif()
     else(LZ4_FOUND)
         file(GLOB LZ4_FILES ${LZ4_LOCAL_DIR}/*.c)
         set(SOURCES ${SOURCES} ${LZ4_FILES})

From d2d7b5d3d6753d80f5b096a26285d73503c5678d Mon Sep 17 00:00:00 2001
From: John Parent <john.parent@kitware.com>
Date: Thu, 10 Oct 2024 16:40:05 -0400
Subject: [PATCH 2/2] use local target for genex

---
 blosc/CMakeLists.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/blosc/CMakeLists.txt b/blosc/CMakeLists.txt
index 55a40ae0..9cc89a06 100644
--- a/blosc/CMakeLists.txt
+++ b/blosc/CMakeLists.txt
@@ -101,8 +101,8 @@ if(NOT DEACTIVATE_LZ4)
             # type of the LZ4 library
             set(LIBS
                 ${LIBS}
-                $<$<STREQUAL:$<TARGET_PROPERTY:Foo,TYPE>,SHARED_LIBRARY>:LZ4::lz4_shared>
-                $<$<NOT:$<STREQUAL:$<TARGET_PROPERTY:Foo,TYPE>,SHARED_LIBRARY>>:LZ4::lz4_static>
+                $<$<STREQUAL:$<TARGET_PROPERTY:TYPE>,SHARED_LIBRARY>:LZ4::lz4_shared>
+                $<$<NOT:$<STREQUAL:$<TARGET_PROPERTY:TYPE>,SHARED_LIBRARY>>:LZ4::lz4_static>
             )
         elseif(SHARED_LIBS OR STATIC_LIBS)
             if(SHARED_LIBS)