[mlpack-svn] r17316 - mlpack/trunk/CMake
fastlab-svn at coffeetalk-1.cc.gatech.edu
fastlab-svn at coffeetalk-1.cc.gatech.edu
Tue Nov 11 12:15:12 EST 2014
Author: rcurtin
Date: Tue Nov 11 12:15:11 2014
New Revision: 17316
Log:
Smarter handling of HDF5 dependency search, especially for Debian systems where
things are Weird(TM).
Modified:
mlpack/trunk/CMake/FindArmadillo.cmake
Modified: mlpack/trunk/CMake/FindArmadillo.cmake
==============================================================================
--- mlpack/trunk/CMake/FindArmadillo.cmake (original)
+++ mlpack/trunk/CMake/FindArmadillo.cmake Tue Nov 11 12:15:11 2014
@@ -222,7 +222,27 @@
# Search for HDF5 (or replacement).
if (NOT "${ARMA_USE_HDF5}" STREQUAL "")
- find_package(HDF5 REQUIRED)
+ find_package(HDF5 QUIET)
+
+ if(NOT HDF5_FOUND)
+ # On Debian systems, the HDF5 package has been split into multiple
+ # packages so that it is co-installable. But this may mean that the
+ # include files are hidden somewhere very odd that the FindHDF5.cmake
+ # script will not find. Thus, we'll also quickly check pkgconfig to see
+ # if there is information on what to use there.
+ find_package(PkgConfig)
+ if (PKG_CONFIG_FOUND)
+ pkg_check_modules(HDF5 hdf5)
+ # But using pkgconfig is a little weird because HDF5_LIBRARIES won't
+ # be filled with exact library paths, like the other scripts. So
+ # instead what we get is HDF5_LIBRARY_DIRS which is the equivalent of
+ # what we'd pass to -L.
+ if (HDF5_FOUND)
+ # I'm not sure what I think of doing this here...
+ link_directories("${HDF5_LIBRARY_DIRS}")
+ endif()
+ endif()
+ endif()
set(SUPPORT_INCLUDE_DIRS "${SUPPORT_INCLUDE_DIRS}" "${HDF5_INCLUDE_DIRS}")
set(SUPPORT_LIBRARIES "${SUPPORT_LIBRARIES}" "${HDF5_LIBRARIES}")
@@ -230,17 +250,38 @@
else("${ARMA_USE_WRAPPER}" STREQUAL "")
# Some older versions still require linking against HDF5 since they did not
- # wrap libhdf5. This was true until 4.300 (check this!).
+ # wrap libhdf5. This was true for versions older than 4.300.
- if(NOT "${ARMA_USE_HDF5}" STREQUAL "")
+ if(NOT "${ARMA_USE_HDF5}" STREQUAL "" AND
+ "${ARMADILLO_VERSION_STRING}" VERSION_LESS "4.300.0")
message(STATUS "Armadillo HDF5 support is enabled and manual linking is "
"required.")
# We have HDF5 support and need to link against HDF5.
- find_package(HDF5 REQUIRED)
+ find_package(HDF5)
- set(SUPPORT_INCLUDE_DIRS "${HDF5_INCLUDE_DIRS}")
- set(SUPPORT_LIBRARIES "${HDF5_LIBRARIES}")
- endif(NOT "${ARMA_USE_HDF5}" STREQUAL "")
+ if(NOT HDF5_FOUND)
+ # On Debian systems, the HDF5 package has been split into multiple
+ # packages so that it is co-installable. But this may mean that the
+ # include files are hidden somewhere very odd that the FindHDF5.cmake
+ # script will not find. Thus, we'll also quickly check pkgconfig to see
+ # if there is information on what to use there.
+ find_package(PkgConfig)
+ if (PKG_CONFIG_FOUND)
+ pkg_check_modules(HDF5 hdf5)
+ # But using pkgconfig is a little weird because HDF5_LIBRARIES won't
+ # be filled with exact library paths, like the other scripts. So
+ # instead what we get is HDF5_LIBRARY_DIRS which is the equivalent of
+ # what we'd pass to -L.
+ if (HDF5_FOUND)
+ # I'm not sure what I think of doing this here...
+ link_directories("${HDF5_LIBRARY_DIRS}")
+ endif()
+ endif()
+
+ set(SUPPORT_INCLUDE_DIRS "${HDF5_INCLUDE_DIRS}")
+ set(SUPPORT_LIBRARIES "${HDF5_LIBRARIES}")
+ endif()
+ endif()
endif("${ARMA_USE_WRAPPER}" STREQUAL "")
More information about the mlpack-svn
mailing list