Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use target_link_libraries instead of ament_target_dependencies #707

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Feb 8, 2025

@ahcorde
Copy link
Contributor

ahcorde commented Feb 10, 2025

Pulls: #707
Gist: https://gist.githubusercontent.com/ahcorde/7e22190d3d79a4a256ae71b682e5f368/raw/94cbe2fedb4f46293ce2767e88b98b8187b29bd7/ros2.repos
BUILD args: --packages-up-to composition demo_nodes_cpp_native dummy_map_server dummy_sensors lifecycle logging_demo pendulum_control --packages-above-and-dependencies composition demo_nodes_cpp_native dummy_map_server dummy_sensors lifecycle logging_demo pendulum_control
TEST args: --packages-select composition demo_nodes_cpp_native dummy_map_server dummy_sensors lifecycle logging_demo pendulum_control --packages-above composition demo_nodes_cpp_native dummy_map_server dummy_sensors lifecycle logging_demo pendulum_control
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/15149

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@ahcorde ahcorde merged commit e869d5b into rolling Feb 10, 2025
3 checks passed
@ahcorde ahcorde deleted the sloretz__ament/ament_cmake#292_remove_ament_target_dependencies branch February 10, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants