Skip to content

Commit

Permalink
Use ament_export_targets for all targets (ros-navigation#4112)
Browse files Browse the repository at this point in the history
* Matches new internal ALIAS targets
* Use ALIAS targets for all internal linkage
* Remove unnecessary calls to ament_target_dependencies in test code
* Export includes in proper folders for overlays in colcon

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
  • Loading branch information
Ryanf55 authored and Marc-Morcos committed Jul 4, 2024
1 parent b4e3bb8 commit c3b9ebf
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions nav2_costmap_2d/test/integration/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -58,12 +58,9 @@ target_link_libraries(test_costmap_publisher_exec
ament_add_gtest_executable(test_costmap_subscriber_exec
test_costmap_subscriber.cpp
)
ament_target_dependencies(test_costmap_subscriber_exec
${dependencies}
)
target_link_libraries(test_costmap_subscriber_exec
nav2_costmap_2d_core
nav2_costmap_2d_client
${PROJECT_NAME}::nav2_costmap_2d_core
${PROJECT_NAME}::nav2_costmap_2d_client
)

ament_add_test(test_collision_checker
Expand Down

0 comments on commit c3b9ebf

Please sign in to comment.