Skip to content

Refactor ament_auto_package usage in CMakeLists.txt#68

Open
XENONFFM wants to merge 1 commit intoautowarefoundation:jazzyfrom
XENONFFM:patch-1
Open

Refactor ament_auto_package usage in CMakeLists.txt#68
XENONFFM wants to merge 1 commit intoautowarefoundation:jazzyfrom
XENONFFM:patch-1

Conversation

@XENONFFM
Copy link

Fixes colcon build warning
Building the package on ros2 jazzy currently prints the following warning:

--- stderr: ros2_socketcan                                
In this package, headers install destination is set to `include` by ament_auto_package. It is recommended to install `include/ros2_socketcan` instead and will be the default behavior of ament_auto_package from ROS 2 Kilted Kaiju. On distributions before Kilted, ament_auto_package behaves the same way when you use USE_SCOPED_HEADER_INSTALL_DIR option.
---

@JWhitleyWork JWhitleyWork changed the base branch from main to jazzy January 21, 2026 00:37
@JWhitleyWork
Copy link
Collaborator

I've re-targted this PR to the jazzy branch. Please rebase your branch on jazzy.

Fixes colcon build warning

Signed-off-by: Finn Scheller <schfin01@students.zhaw.ch>
@XENONFFM
Copy link
Author

@JWhitleyWork sorry for the delay, should be done now

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