Use target_link_libraries instead of ament_target_dependencies (#325)

* Use target_link_libraries instead of ament_target_dependencies

Signed-off-by: Antón Casas <[email protected]>

* Remove ament_target_dependencies from test/CmakeLists.txt

Signed-off-by: Antón Casas <[email protected]>

---------

Signed-off-by: Antón Casas <[email protected]>

Author: 4ntn

rmw_microxrcedds_c/CMakeLists.txt

@@ -246,8 +246,8 @@ if(RMW_UXRCE_GRAPH)
     PROPERTY
       IMPORTED_LOCATION ${micro_ros_msgs_LIBRARIES__rosidl_typesupport_microxrcedds_c})
 
-  ament_target_dependencies(${PROJECT_NAME}
-    "micro_ros_msgs"
+  target_link_libraries(${PROJECT_NAME}
+    ${micro_ros_msgs_TARGETS}
 )
 endif()
 
@@ -260,8 +260,8 @@ target_link_libraries(${PROJECT_NAME}
 # Type support lock-up mechanism
 find_package(rosidl_typesupport_microxrcedds_c)
 if(rosidl_typesupport_microxrcedds_c_FOUND)
-  ament_target_dependencies(${PROJECT_NAME}
-    "rosidl_typesupport_microxrcedds_c"
+  target_link_libraries(${PROJECT_NAME}
+    rosidl_typesupport_microxrcedds_c::rosidl_typesupport_microxrcedds_c
 )
   target_compile_definitions(${PROJECT_NAME}
     PUBLIC
@@ -271,19 +271,19 @@ else()
   find_package(rosidl_typesupport_microxrcedds_cpp REQUIRED)
 endif()
 if(rosidl_typesupport_microxrcedds_cpp_FOUND)
-  ament_target_dependencies(${PROJECT_NAME}
-    "rosidl_typesupport_microxrcedds_cpp"
+  target_link_libraries(${PROJECT_NAME}
+    rosidl_typesupport_microxrcedds_cpp::rosidl_typesupport_microxrcedds_cpp
 )
   target_compile_definitions(${PROJECT_NAME}
     PUBLIC
       HAVE_CPP_TYPESUPPORT
 )
 endif()
 
-ament_target_dependencies(${PROJECT_NAME}
-  "rcutils"
-  "rmw"
-  "microxrcedds_client"
+target_link_libraries(${PROJECT_NAME}
+  rcutils::rcutils
+  rmw::rmw
+  microxrcedds_client
 )
 
 configure_rmw_library(${PROJECT_NAME})

rmw_microxrcedds_c/test/CMakeLists.txt

@@ -44,7 +44,7 @@ macro(rmw_test TEST_NAME TEST_SOURCE)
     CXX_STANDARD_REQUIRED
       YES)
 
-  ament_target_dependencies(${TEST_NAME} rmw)
+  target_link_libraries(${TEST_NAME} rmw::rmw)
 endmacro()
 
 rmw_test(test-node        test_node.cpp)