moveit/moveit_msgs

Move some other *_msgs packages into moveit_msgs repo

130s opened this issue · 2 comments

130s commented

household_objects_database_msgs and manipulation_msgs are dependended by MoveIt! AFAIK. Both of those packages are maintained in a separate repo as a unary package, which duplicated the effort upon making a release. Since the primary usage of those packages might be MoveIt! these days, it makes sense to move them into MoveIt!-related repo and combine the maintenance effort.

Caveat is, all packages in a single repo needs to align the same version AFAIK. We can adjust to the highest available version as 0.9.x without any issue IMO.

  • 0.1.2 household_objects_database_msgs
  • 0.2.1 manipulation_msgs
  • 0.9.1 (kinetic-devel) moveit_msgs
v4hn commented

Both packages are not (or at least should not be) dependencies of MoveIt kinetic+.
See moveit/moveit@2956439
What makes you think we still need them? Did I miss something in the referenced commit?

130s commented

No you didn't, you're right. I also left a comment at ros-interactive-manipulation/manipulation_msgs#10 (comment) Closing as this is unnecessary.