Fix build for melodic (#33)

iris-ua · Sep 8, 2022 · 454dfc5 · 454dfc5
1 parent b01dbe6
commit 454dfc5
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -17,13 +17,6 @@ find_package(Eigen3 3.3 REQUIRED NO_MODULE)
 # the code
 add_subdirectory(src)
 
-# libs and binaries, generate export targets
-install(TARGETS iris_lama
-  EXPORT ${CMAKE_PROJECT_NAME}Targets
-  ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
-  LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
-  RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
-
 # header files, note the "/" after include
 install(DIRECTORY include/
   DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -67,3 +67,9 @@ endif()
 target_link_libraries(iris_lama PUBLIC pthread Eigen3::Eigen)
 target_include_directories(iris_lama PRIVATE extern/minisam)
 
+# libs and binaries, generate export targets
+install(TARGETS iris_lama
+  EXPORT ${CMAKE_PROJECT_NAME}Targets
+  ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
+  LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
+  RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
diff --git a/vendor/minisam/CMakeLists.txt b/vendor/minisam/CMakeLists.txt
@@ -25,4 +25,4 @@ set(minisam_SRC
 )
 
 add_library(minisam OBJECT ${minisam_SRC})
-target_link_libraries(minisam PRIVATE Eigen3::Eigen)
+target_include_directories(minisam PRIVATE ${EIGEN3_INCLUDE_DIR})