Skip to content

Commit

Permalink
chore: fix missing dependencies that cause CI fail
Browse files Browse the repository at this point in the history
  • Loading branch information
mojomex committed Aug 30, 2024
1 parent bc9a811 commit c2dbba8
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
2 changes: 2 additions & 0 deletions nebula_tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ find_package(nebula_common REQUIRED)
find_package(nebula_decoders REQUIRED)
find_package(PCL REQUIRED COMPONENTS common)
find_package(rosbag2_cpp REQUIRED)
find_package(diagnostic_updater REQUIRED)


if(BUILD_TESTING)
Expand Down Expand Up @@ -53,6 +54,7 @@ if(BUILD_TESTING)

set(CONTINENTAL_TEST_LIBRARIES
${NEBULA_TEST_LIBRARIES}
${diagnostic_updater_TARGETS}
nebula_decoders::nebula_decoders_continental
)

Expand Down
2 changes: 2 additions & 0 deletions nebula_tests/continental/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ add_library(continental_ros_decoder_test_ars548 SHARED

target_include_directories(continental_ros_decoder_test_ars548 PUBLIC
${NEBULA_TEST_INCLUDE_DIRS}
${diagnostic_updater_INCLUDE_DIRS}
)

target_link_libraries(continental_ros_decoder_test_ars548
Expand All @@ -32,6 +33,7 @@ add_library(continental_ros_decoder_test_srr520 SHARED

target_include_directories(continental_ros_decoder_test_srr520 PUBLIC
${NEBULA_TEST_INCLUDE_DIRS}
${diagnostic_updater_INCLUDE_DIRS}
)

target_link_libraries(continental_ros_decoder_test_srr520
Expand Down
1 change: 1 addition & 0 deletions nebula_tests/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
<buildtool_depend>ament_cmake_auto</buildtool_depend>
<buildtool_depend>ros_environment</buildtool_depend>

<depend>diagnostic_updater</depend>
<depend>nebula_common</depend>
<depend>nebula_decoders</depend>
<depend>rosbag2_cpp</depend>
Expand Down

0 comments on commit c2dbba8

Please sign in to comment.