Skip to content

Replace ament_target_dependencies with target_link_libraries#82

Merged
Yadunund merged 2 commits intomainfrom
ahcorde/rolling/replace_ament_target_dependencies
May 23, 2025
Merged

Replace ament_target_dependencies with target_link_libraries#82
Yadunund merged 2 commits intomainfrom
ahcorde/rolling/replace_ament_target_dependencies

Conversation

@ahcorde
Copy link
Copy Markdown
Contributor

@ahcorde ahcorde commented May 22, 2025

ament_target_dependencies is deprecated on kilted and removed in rolling, It will require a new release on rolling.

Debs are broken https://build.ros2.org/view/Rbin_uN64/job/Rbin_uN64__rmf_visualization_navgraphs__ubuntu_noble_amd64__binary/

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested review from Yadunund and mxgrey May 22, 2025 11:42
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from Yadunund May 22, 2025 22:03
Copy link
Copy Markdown
Contributor

@mxgrey mxgrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the migration!

@Yadunund Yadunund merged commit 8443640 into main May 23, 2025
2 checks passed
@Yadunund Yadunund deleted the ahcorde/rolling/replace_ament_target_dependencies branch May 23, 2025 16:15
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.

3 participants