Use ament_export_targets for all targets (ros-navigation#4112)

Ryanf55 · Marc-Morcos · commit 7d158b4dda24 · 2024-07-04T12:04:16.000-04:00
* 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 &lt;ryanfriedman5410+github@gmail.com&gt;
diff --git a/nav2_costmap_2d/test/integration/CMakeLists.txt b/nav2_costmap_2d/test/integration/CMakeLists.txt
@@ -66,12 +66,9 @@ target_link_libraries(test_costmap_subscriber_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

Original file line number	Diff line number	Diff line change
`@@ -66,12 +66,9 @@ target_link_libraries(test_costmap_subscriber_exec`
`66`	`66`	`ament_add_gtest_executable(test_costmap_subscriber_exec`
`67`	`67`	`test_costmap_subscriber.cpp`
`68`	`68`	`)`
`69`		`-ament_target_dependencies(test_costmap_subscriber_exec`
`70`		`- ${dependencies}`
`71`		`-)`
`72`	`69`	`target_link_libraries(test_costmap_subscriber_exec`
`73`		`- nav2_costmap_2d_core`
`74`		`- nav2_costmap_2d_client`
	`70`	`+ ${PROJECT_NAME}::nav2_costmap_2d_core`
	`71`	`+ ${PROJECT_NAME}::nav2_costmap_2d_client`
`75`	`72`	`)`
`76`	`73`
`77`	`74`	`ament_add_test(test_collision_checker`