moveit / moveit_core

THIS REPO HAS MOVED TO https://github.com/ros-planning/moveit
31 stars 76 forks source link

catkin_lint cleanup jade-devel #317

Closed v4hn closed 8 years ago

v4hn commented 8 years ago

This is basically #305 for jade.

davetcoleman commented 8 years ago

why is travis not running?

v4hn commented 8 years ago

No idea, it works for #318 . Any idea how to trigger it?

d130s commented 8 years ago

It can happen between Travis and Github. Close the pr and reopen it might work (I can't for now from my phone).

davetcoleman commented 8 years ago

It doesn't appear on Travis... maybe it is to similar to #318 so it filtered it out? Attempting to reopen.

davetcoleman commented 8 years ago

Travis: Could not find a package configuration file provided by "Eigen3"

v4hn commented 8 years ago

okok, I yield :p . I thought find_package(Eigen3) should be fine in jade, but this still hits this weird problem that catkin tools can't find Eigen3 although FindEigen3.cmake is around in ubuntu 14.04.. Anyway, I updated the request.