ros2 / rosidl_typesupport

Packages which provide the typesupport for ROS messages and services
Apache License 2.0
13 stars 34 forks source link

Remove direct dependencies on vendor typesupport packages. #26

Closed nuclearsandwich closed 6 years ago

nuclearsandwich commented 6 years ago

This change will reduce the amount of effort required to ensure that vendor typesupport packages are optional for end-users. And it cleans up our dependency tree a little with caveats*

The vendor typesupport packages are also members in the respective groups rosidl_typesupport_c{,pp}_packages and thus will still be directly depended upon by this package for source builds.

Here's a graph of the tree from an example message package (std_msgs) to the vendor typesupport packages.

*These graphs do not consider group dependencies which I think re-introduce the "overspecified" dependencies from the before image below.

Before: deps before


After: deps after

nuclearsandwich commented 6 years ago