Use CMAKE_PREFIX_PATH to pass Homebrew install location - Passing HINTS disables the MODULE mode of find_package() which precludes users from creating their own find modules to provide Ceres' dependencies. Change-Id: I6f2edf429331d13fe67bf61ac4b79d17579d9a57
diff --git a/CMakeLists.txt b/CMakeLists.txt index 33e98b1..33d7e76 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt
@@ -81,7 +81,7 @@ OUTPUT_STRIP_TRAILING_WHITESPACE) message(STATUS "Detected Homebrew with install prefix: " "${HOMEBREW_INSTALL_PREFIX}, adding to CMake search paths.") - list(APPEND HOMEBREW_INCLUDE_DIR_HINTS "${HOMEBREW_INSTALL_PREFIX}/include") + list(APPEND CMAKE_PREFIX_PATH "${HOMEBREW_INSTALL_PREFIX}") endif() endif() @@ -239,7 +239,7 @@ # Eigen. # Eigen delivers Eigen3Config.cmake since v3.3.3 -find_package(Eigen3 3.3 REQUIRED HINTS ${HOMEBREW_INCLUDE_DIR_HINTS}) +find_package(Eigen3 3.3 REQUIRED) if (EIGEN3_FOUND) message("-- Found Eigen version ${EIGEN3_VERSION_STRING}: ${EIGEN3_INCLUDE_DIRS}") if (CMAKE_SYSTEM_PROCESSOR MATCHES "^(aarch64.*|AARCH64.*)" AND @@ -390,7 +390,7 @@ # GFlags. if (GFLAGS) # Don't search with REQUIRED as we can continue without gflags. - find_package(gflags 2.2.0 HINTS "${HOMEBREW_INCLUDE_DIR_HINTS}") + find_package(gflags 2.2.0) if (gflags_FOUND) if (TARGET gflags) message("-- Found Google Flags (gflags) version ${gflags_VERSION}: ${gflags_DIR}")