Use system Eigen3 CMake Config and Eigen target when linking (#99)

ethz-adrl · Sep 14, 2023 · 3f302dd · 3f302dd
1 parent c914d89
commit 3f302dd
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 367 deletions.
diff --git a/ifopt-config.cmake b/ifopt-config.cmake
@@ -35,6 +35,7 @@
 #
 #=============================================================================
 
+find_dependency(Eigen3)
 
 # these are autogenerate by cmake
 include("${CMAKE_CURRENT_LIST_DIR}/ifopt_core-targets.cmake")

diff --git a/ifopt_core/CMakeLists.txt b/ifopt_core/CMakeLists.txt
@@ -1,4 +1,3 @@
-set(CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake" ${CMAKE_MODULE_PATH})
 find_package(Eigen3 REQUIRED)
 
 
@@ -15,7 +14,10 @@ target_include_directories(${LIB_CORE}
     $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
     $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/test>
     $<INSTALL_INTERFACE:include>
-    ${EIGEN3_INCLUDE_DIR}
+)
+target_link_libraries(${LIB_CORE} 
+  PUBLIC
+    Eigen3::Eigen
 )
 # better specify features you need, not c++ version
 target_compile_features(${LIB_CORE} 

diff --git a/ifopt_core/cmake/FindEigen3.cmake b/ifopt_core/cmake/FindEigen3.cmake
diff --git a/ifopt_core/cmake/FindGTest.cmake b/ifopt_core/cmake/FindGTest.cmake