Merge pull request #926 from paulromano/hdf5-transitive-dependencies

Do not link against several transitive dependencies of HDF5
svalinn · Dec 5, 2023 · 1186825 · 1186825
2 parents 8b98988 + 65fbe97
commit 1186825
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/cmake/FindMOAB.cmake b/cmake/FindMOAB.cmake
@@ -22,6 +22,8 @@ include(${MOAB_CMAKE_CONFIG})
 set(ENV{PATH} "${HDF5_DIR}:$ENV{PATH}")
 set(CMAKE_FIND_LIBRARY_SUFFIXES ${CMAKE_SHARED_LIBRARY_SUFFIX})
 find_package(HDF5 REQUIRED)
+# Remove HDF5 transitive dependencies that are system libraries
+list(FILTER HDF5_LIBRARIES EXCLUDE REGEX ".*lib(pthread|dl|m).*")
 set(HDF5_LIBRARIES_SHARED ${HDF5_LIBRARIES})
 # CMake doesn't let you find_package(HDF5) twice so we have to do this instead
 if (BUILD_STATIC_LIBS)

diff --git a/doc/CHANGELOG.rst b/doc/CHANGELOG.rst
@@ -35,6 +35,7 @@ Next version
    * Tweak conda environment for Windows build to avoid conflicting gtest headers (#888)
    * Restrict cython version for MOAB (#893)
    * Various documentation updates (#869)
+   * Fix for HDF5 library linking issue (#926)
 
 v3.2.2
 ====================