ament / ament_index

Apache License 2.0
13 stars 27 forks source link

Install includes to include/${PROJECT_NAME} #83

Closed sloretz closed 2 years ago

sloretz commented 2 years ago

Part of ros2/ros2#1150

This installs includes to include/${PROJECT_NAME} to mitigate include directory search order issues when overriding packages in desktop.

Marked as draft because I'm opening a bunch of PRs at once to run CI all at the same time.

sloretz commented 2 years ago

Linking all the PRs with the same branch name here:

sloretz commented 2 years ago

Looks good to me with green CI. Feel free to add me as a reviewer on the related PRs as they come out of draft state.

It looks like some of the packages will take longer, so I'll do CI with a repos file

CI (repos file build: --packages-above-and-dependencies ament_index_cpp test: --packages-select ament_index_cpp)