DLu/roscompile

Consider integration with catkin lint?

koenlek opened this issue · 2 comments

We use "catkin lint" to check for errors in our package.xml & CMakeLists.txt files.

It can be found here: https://github.com/fkie/catkin_lint
It is part of the official ros packages repository (http://packages.ros.org/ros/ubuntu).

Wouldn't it be a good idea to explore integrating these tools? Having the logic of these tools in the same code would make sense, first the logic to find the error and next a procedure to correct for it.

DLu commented

Integrating with catkin lint was brought up after I initially announced the tool. Without having looked into catkin_lint too much, what interfaces are available for integrating?

130s commented

catkin_lint is, as the name indicates, for catkin packages, which can be ROS-agnostic, while I see roscompile does a bunch of ROS-specific stuff. IMO coupling too many frameworks would limit the usecases (and that's been getting more important for me having worked in a team where ROS is just one of the frameworks we heavily rely on, not a whole).

I see the benefit of merging 2 tools (less maintenance, potentially more user-base etc.)

Just my 1:coin: