gazebo-tooling / release-tools

8 stars 9 forks source link

Audit Gazebo libraries use of CMake #833

Open mjcarroll opened 1 year ago

mjcarroll commented 1 year ago

A tracking issue to make sure that all of the gazebo libraries can do the following:

scpeters commented 1 year ago

I think we could make GitHub workflows for this?

Work in a colcon workspace without --merge-install

is this in a workspace with other packages or all by itself?

mjcarroll commented 1 year ago

is this in a workspace with other packages or all by itself?

Both. In my mind, this insures that we aren't relying on transitive include paths. It should potentially help catch cases where those things are misconfigured.

scpeters commented 1 year ago

is this in a workspace with other packages or all by itself?

Both. In my mind, this insures that we aren't relying on transitive include paths. It should potentially help catch cases where those things are misconfigured.

so to avoid combinatorial explosion, would the following two cases for each package be enough?