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} #670

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

sloretz
Copy link
Contributor

@sloretz sloretz commented Mar 15, 2022

To be merged at the same time as ros2/rosidl_typesupport_fastrtps#88

Requires #662
Part of ros2/ros2#1150

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

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
@sloretz sloretz changed the base branch from sloretz__misc_generator_cleanup to master March 24, 2022 23:35
@sloretz sloretz force-pushed the sloretz__rosidl_pipeline__include_project_name branch from 9720af8 to 3781117 Compare March 24, 2022 23:36
@sloretz
Copy link
Contributor Author

sloretz commented Mar 24, 2022

CI for this and ros2/rosidl_typesupport_fastrtps#88

  • build: all
  • test: --packages-select rosidl_generator_c rosidl_generator_cpp rosidl_typesupport_introspection_c rosidl_typesupport_introspection_cpp rosidl_typesupport_fastrtps_c rosidl_typesupport_fastrtps_cpp

Jobs

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

@sloretz sloretz merged commit c4d31b3 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