Treat Eigen as a system-wide dependency on Unix

Eigen headers are now located using pkg_check_modules() rather than using the version in external/eigen.
codereader · Apr 11, 2021 · b32d7f7 · b32d7f7
1 parent b68926f
commit b32d7f7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -68,6 +68,7 @@ pkg_check_modules(VORBIS vorbisfile REQUIRED)
 pkg_check_modules(X11 x11 REQUIRED)
 pkg_check_modules(ZLIB zlib REQUIRED)
 pkg_check_modules(GLIB glib-2.0 REQUIRED)
+pkg_check_modules(EIGEN eigen3 REQUIRED)
 
 # Locate wxWidgets
 find_package(wxWidgets REQUIRED
@@ -78,7 +79,7 @@ include(${wxWidgets_USE_FILE})
 find_package(Python REQUIRED COMPONENTS Development)
 
 # Global includes and flags
-include_directories(libs libs/libfmt external/eigen include)
+include_directories(libs libs/libfmt include)
 add_compile_definitions(POSIX
                         WXINTL_NO_GETTEXT_MACRO
                         FMT_HEADER_ONLY

diff --git a/libs/math/CMakeLists.txt b/libs/math/CMakeLists.txt
@@ -1,2 +1,3 @@
 add_library(math
             AABB.cpp Frustum.cpp Matrix4.cpp Plane3.cpp)
+target_compile_options(math PUBLIC ${EIGEN_CFLAGS})