Skip to content

Commit

Permalink
use different cmake variable for eigen include dir
Browse files Browse the repository at this point in the history
Eigen3.2's upstream FindEigen3.cmake specifies only
EIGEN3_INCLUDE_DIR, not EIGEN3_INCLUDE_DIRS:
https://bitbucket.org/eigen/eigen/src/cafa6d909f409f4ec33070ebf991a77e7b747dc1/cmake/FindEigen3.cmake?at=default&fileviewer=file-view-default#FindEigen3.cmake-10

Eigen3.3-beta provides an Eigen3Config that defines
EIGEN3_INCLUDE_DIRS AND _DIR, but that's still beta, even though
Ubuntu introduced it in xenial already. So to stay backwards
compatible, _INCLUDE_DIR should be used.
  • Loading branch information
Michael Görner committed Jun 9, 2016
1 parent b12495e commit 9fdef94
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion kinect2_registration/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ endif()

if(Eigen3_FOUND)
message(STATUS "CPU based depth registration enabled")
include_directories(${EIGEN3_INCLUDE_DIRS})
include_directories(${EIGEN3_INCLUDE_DIR})
set(DEPTH_REG_CPU ON)
add_definitions(-DDEPTH_REG_CPU)
else()
Expand Down

0 comments on commit 9fdef94

Please sign in to comment.