-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Catkin Lint #10
Comments
FTI catkin_lint : https://github.com/fkie/catkin_lint |
Awesome. I was just about to try that as well. And it looks like it will really help On 14 March 2014 14:22, Jihoon Lee [email protected] wrote:
Phone : +82-10-5400-3296 (010-5400-3296) |
Does it have a cmake api? |
It is described in his github document. https://github.com/fkie/catkin_lint Copied from his README. Catkin Build Integration It is recommended to run catkin_lint at workspace configuration time. The simplest way is to symlink /opt/ros/$ROS_DISTRO/share/catkin/cmake/toplevel.cmake to your $WSDIR/src folder manually (do not use catkin_init_workspace). Then add the following CMakeLists.txt:
|
Ah. Thats awkward. He needs to make a cmake api like I do for rosjava and
|
He would be happy to accept pull request :) |
I'm already contributing enough everywhere and this one isn't high priority
On 19 March 2014 11:43, Jihoon Lee [email protected] wrote:
Phone : +82-10-5400-3296 (010-5400-3296) |
Remaining lint jobs
|
Have tested catkin_lint with all rocon packages.
The text was updated successfully, but these errors were encountered: