Indigo branch and release missing pcl_ros dependency

atenpas / handle_detector

ROS package to localize handles in 3D point clouds

BSD 2-Clause "Simplified" License

25 stars 6 forks source link

Indigo branch and release missing pcl_ros dependency #4

Open mryellow opened 9 years ago

mryellow commented 9 years ago

CMakeLists.txt

Add pcl_ros to find_package and catkin_package.
Remove PCL from catkin_package->DEPENDS.
Remove ${PCL_INCLUDE_DIRS} from include_directories

package.xml

Add pcl_ros depend tags.

edit: Have had this come up before with another package, something unique about my debian:jessie mixed source install. Probably works for everyone else, but can't hurt specifying the dependency.

atenpas commented 9 years ago

I had just removed the PCL ROS dependencies and instead linked to PCL directly because I thought that this was causing a build error on jenkins. Not sure what the problem is there. I will add your suggestions to the next release. Thanks!

mryellow commented 9 years ago

Well it worked for me.... but I may be an edge-case, debian:jessie took a fair few source installs to get working.