ros-tooling/libstatistics_collector

libstatistics_collector has unnecessary dependency on `std_msgs`

clalancette opened this issue · 0 comments

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.