Catch dependency errors building in isolation (#70)

* [converters] add missing ecl_exceptions dependency * [linear_algebra] sophus has an interface ...
stonier · May 4, 2018 · 3b29a7b · 3b29a7b
1 parent 8766a54
commit 3b29a7b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/ecl_converters/CMakeLists.txt b/ecl_converters/CMakeLists.txt
@@ -44,6 +44,7 @@ target_link_libraries(
     ecl_concepts::ecl_concepts
     ecl_config::ecl_config
     ecl_errors::ecl_errors
+    ecl_exceptions::ecl_exceptions
     ecl_mpl::ecl_mpl
     ecl_type_traits::ecl_type_traits
 )

diff --git a/ecl_converters/src/examples/CMakeLists.txt b/ecl_converters/src/examples/CMakeLists.txt
@@ -3,5 +3,7 @@
 ##############################################################################
 
 add_executable(demo_float_converters float_converters.cpp)
-target_link_libraries(demo_float_converters ${PROJECT_NAME})
+target_link_libraries(demo_float_converters
+  ${PROJECT_NAME}
+)
 install(TARGETS demo_float_converters RUNTIME DESTINATION lib/${PROJECT_NAME})
diff --git a/ecl_linear_algebra/src/lib/CMakeLists.txt b/ecl_linear_algebra/src/lib/CMakeLists.txt
@@ -15,7 +15,6 @@ target_include_directories(
   INTERFACE
     $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
     $<INSTALL_INTERFACE:include>
-    $<INSTALL_INTERFACE:${Sophus_INCLUDE_DIRS}>
 )
 
 target_link_libraries(
@@ -25,6 +24,7 @@ target_link_libraries(
     ecl_exceptions::ecl_exceptions
     ecl_formatters::ecl_formatters
     ecl_math::ecl_math
+    Sophus::Sophus
 )
 
 set_target_properties(${PROJECT_NAME}