Install includes to include/ and misc CMake fixes#225
Install includes to include/ and misc CMake fixes#225clalancette merged 2 commits intoros-drivers:ros2from
Conversation
|
@ros-pull-request-builder retest this please |
|
We're actually going to have to split this up into several different branches, since we only want to land this change on |
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
5348737 to
e4ff192
Compare
|
I've rebased this onto the latest, which should 🤞 make CI pass here now. |
Make sure to really declare and find all dependencies. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
|
@sloretz I just pushed 0d2b8e5 here, which adds in some additional infrastructure that the CMake and package.xml files were missing. This now looks good to me, and passes CI, so I'm going to approve it. However, since I did enough additional work here, I'd appreciate a comment from you on whether you agree with the changes I've done before I merge this. |
|
I'm going to go ahead with this one so we can get a release out for Rolling and try to fix the spacenav regression. If anyone wants to do a review after merge, I'm happy to make follow-up changes. |
Part of ros2/ros2#1150
This installs includes to
include/${PROJECT_NAME}to mitigate include directory search order issues when overriding packages in desktop.Part of ament/ament_cmake#292
This replaces
ament_target_dependencies()calls withtarget_link_libraries().I also made changes to use modern CMake targets, notably by creating IMPORTED targets
wiimote::bluetoothandwiimote::cwidto make thewiimote::wiimote_libtarget exportable.Requires ros2/rclcpp#1855 for the
rclcpp_components::componenttarget.