add rosdep pcl_conversions to prevent pointcloud_screenpoint_nodelet error(groovy/rosbuild)

[aginika]

When I rosmake the jsk_pcl_ros in groovy/rosbuild, I got the below error.
[ 82%] Building CXX object CMakeFiles/jsk_pcl_ros.dir/src/pointcloud_screenpoint_nodelet.cpp.o
/home/*******/ros/groovy/jsk-ros-pkg/jsk_recognition/jsk_pcl_ros/src/pointcloud_screenpoint_nodelet.cpp:2:45: fatal error: pcl_conversions/pcl_conversions.h: No such file or directory
compilation terminated.

[k-okada]

Why did travis passed the test?
Jenkins also said it was OK.
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/jsk.rosbuild/55/

[k-okada]

I suppose @aginika is forget to install pcl-conversions, please confirm that ros-groovy-pcl-conversions is installed and if not,
run https://github.com/jsk-ros-pkg/jsk_common/blob/master/rosdep-update.sh to install them. Just running rosdep udpate will not work since jsk-ros-pkg contains both manifest.xml and package.xml.

see jsk-ros-pkg/jsk_common#301 for discussion.

[aginika]

Yes... I forgot to run https://github.com/jsk-ros-pkg/jsk_common/blob/master/rosdep-update.sh .
Please revert it if you can.

[garaemon]

I think it's not so bad to add pcl_conversions to manifest.xml.
However we should add depemd tag as well as rosdepmtag

[k-okada]

Yes, I think correct manifest is

  <depend package="pcl" />
  <depend package="pcl_conversions" />
  <depend package="pcl_ros" />
  <depend package="opencv2" />
  <depend package="tf" />
  <depend package="sensor_msgs" />
  <depend package="geometry_msgs" />
  <depend package="stereo_msgs" />
  <depend package="nodelet" />
  <depend package="dynamic_reconfigure" />
  <depend package="std_srvs" />
  <depend package="std_msgs" />
  <depend package="image_transport" />
  <depend package="image_geometry" />
  <depend package="image_view2" />
  <depend package="eigen_conversions" />

  <rosdep name="tbb" />

[k-okada]

@aginika could you create one more PR on manifest.xml, which I suggested in #30 (comment)?

Thanks in advance.