libstatistics_collector has unnecessary dependency on `std_msgs`
clalancette opened this issue · 0 comments
clalancette commented
Description
Looking at https://github.com/ros-tooling/libstatistics_collector/blob/bb824c52e45e0e224046008e2de5f86d9eb71c5f/package.xml, there is currently a build_depend and exec_depend on the std_msgs package. However, the core functionality of this package does not depend on std_msgs; only the tests do.
Expected Behavior
No build or run time dependency on std_msgs, but a test time dependency on std_msgs.
Actual Behavior
Build and run time dependency on std_msgs.
System (please complete the following information)
- OS: Ubuntu 22.04
- ROS 2 Distro: Rolling
Additional context
I tried to briefly fix this in #54 , but CI failed due to some kind of dependency cycle that I didn't expect/wasn't aware of. Since that was a side issue for that PR, I decided to punt on it and open this issue instead, which we should investigate separately.