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

Install generated headers to include/${PROJECT_NAME} #88

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Mar 15, 2022

Requires #87
Part of ros2/ros2#1150
To be merged at the same time as ros2/rosidl#670

This installs generated headers to a unique include directory to prevent include directory search order issues when overriding packages.

@sloretz sloretz self-assigned this Mar 15, 2022
@sloretz sloretz marked this pull request as draft March 22, 2022 20:35
@delete-merged-branch delete-merged-branch bot deleted the branch master March 24, 2022 20:21
@sloretz sloretz marked this pull request as ready for review March 24, 2022 20:21
@sloretz sloretz changed the base branch from sloretz__rosidl_typesupport_fastrtps_cmake_issues to master March 24, 2022 23:38
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz force-pushed the sloretz__rosidl_pipeline__include_project_name branch from 2fe9d3a to 53c434b Compare March 24, 2022 23:39
@sloretz sloretz merged commit 0ab6b0c into master Mar 28, 2022
@delete-merged-branch delete-merged-branch bot deleted the sloretz__rosidl_pipeline__include_project_name branch March 28, 2022 16:53
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