Install includes to include/${PROJECT_NAME}

[sloretz]

Also, use SKIP_INSTALL on interfaces generated just for testing to prevent those headers from being installed.

Part of ros2/ros2#1150 - this avoids include directory search order issues when overriding this package.

[jhdcs]

This looks good to me.

Idle thought/question: Is there any way we can set up automatic CI testing for this? Just in case of future regressions (though CMake doesn't tend to change often, so that might never happen. Knock on wood)

[sloretz]

Idle thought/question: Is there any way we can set up automatic CI testing for this? Just in case of future regressions (though CMake doesn't tend to change often, so that might never happen. Knock on wood)

Automatic CI tests for the test headers being installed, or to make sure all includes are installed to an extra project name directory, or something else? I don't have any good ideas for how to test the first two, but CI with downstream packages included makes sure the include search directory exported by CMake matches the one installed.

[sloretz]

CI (repos file build: --packages-up-to libstatistics_collector rclcpp test: --packages-select libstatistics_collector)

• Linux
• Linux-aarch64
• Windows