search

moveit / moveit_ikfast

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

Missing builddep on cmake_modules in generated ikfast packages #27

Closed gavanderhoorn closed 10 years ago

gavanderhoorn commented 10 years ago

Running catkin_lint on a recently (re)generated ikfast plugin package:

X_moveit_plugins: CMakeLists.txt(4): warning: find_package(Eigen) before find_package(cmake_modules)

This makes sense: FindEigen.cmake is part of the cmake_modules package, which should be find_package()-ed first. It should then also become a build_depend of the plugin package.

davetcoleman commented 10 years ago

Can you make the fix in this file? https://github.com/ros-planning/moveit_ikfast/blob/master/templates/CMakeLists.txt Thanks!