Merge pull request #7 from Robotic-Decision-Making-Lab/bug-devcontainer

Revert changes to CMakeLists file to support backwards compatibility
Robotic-Decision-Making-Lab · Sep 23, 2024 · ab0d8d1 · ab0d8d1
2 parents 7e8010f + f03bb68
commit ab0d8d1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
@@ -1,9 +1,9 @@
 {
-  "name": "ROS 2 NVIDIA Dev Container",
+  "name": "Dev Container",
   "dockerFile": "Dockerfile",
   "context": "../",
-  "workspaceMount": "source=${localWorkspaceFolder},target=/home/ubuntu/ws_ros/src/auv_controllers,type=bind",
-  "workspaceFolder": "/home/ubuntu/ws_ros/src/auv_controllers",
+  "workspaceMount": "source=${localWorkspaceFolder},target=/home/ubuntu/ws_ros/src/hydrodynamics,type=bind",
+  "workspaceFolder": "/home/ubuntu/ws_ros/src/hydrodynamics",
   "remoteUser": "ubuntu",
   "customizations": {
     "vscode": {

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -13,24 +13,26 @@ add_library(hydrodynamics SHARED)
 target_sources(hydrodynamics
   PRIVATE
     src/hydrodynamics.cpp
+)
+target_include_directories(hydrodynamics
   PUBLIC
-    FILE_SET HEADERS
-    BASE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}/include
-    FILES
-      ${CMAKE_CURRENT_SOURCE_DIR}/include/hydrodynamics/hydrodynamics.hpp
-      ${CMAKE_CURRENT_SOURCE_DIR}/include/hydrodynamics/eigen.hpp
+    $<INSTALL_INTERFACE:include/hydrodynamics>
+    $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
 )
-
 target_compile_features(hydrodynamics PUBLIC cxx_std_20)
 target_link_libraries(hydrodynamics Eigen3::Eigen)
 
+install(
+  DIRECTORY include/
+  DESTINATION include/hydrodynamics
+)
+
 install(
   TARGETS hydrodynamics
   EXPORT export_hydrodynamics
   RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
   LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
   ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
-  FILE_SET HEADERS
 )
 
 if(BUILD_TESTING)