Remove PCL dependency

[vrabaud]

No description provided.

[sachinchitta]

Correct license information

[isucan]

+1
Vincent, thanks for the patch. Should the iterator stuff be a separate package?

[vrabaud]

ok, I fixed the licenses.
Those files are in sensor_msgs in Indigo but will not be backported to hydro according to ros/common_msgs#16, hence the copy.

[davetcoleman]

@vrabaud it looks like the repo is out of sync with this PR, could you try updating it to the latest hydro-devel?

is there some other reason this has not been merged yet?

[vrabaud]

Hey, the Travis job passed. Now, I copied the PointCloud2 iterators over so that it also works in Hydro. I see that you have branched moveit_core for Indigo. Are you going to do the same for moveit_ros ? In which case I would simpify my PR and only make it for Indigo.

[isucan]

I think we should branch for indigo.
On Sep 7, 2014 12:31 PM, "Vincent Rabaud" notifications@github.com wrote:

Hey, the Travis job passed. Now, I copied the PointCloud2 iterators over
so that it also works in Hydro. I see that you have branched moveit_core
for Indigo. Are you going to do the same for moveit_ros ? In which case I
would simpify my PR and only make it for Indigo.

—
Reply to this email directly or view it on GitHub
#449 (comment)
.

[vrabaud]

Then please do and I'll make my PR there. Thx.

[davetcoleman]

I have branched moveit_ros for indigo as requested:
https://github.com/ros-planning/moveit_ros/tree/indigo-devel

Now that moveit_core, moveit_ros, and moveit_planners are branched, I vote we set them as the default github branches. Anyone opposed?

[isucan]

+1 for default branch
On Sep 8, 2014 6:03 PM, "Dave Coleman" notifications@github.com wrote:

I have branched moveit_ros for indigo as requested:
https://github.com/ros-planning/moveit_ros/tree/indigo-devel

Now that moveit_core, moveit_ros, and moveit_planners are branched, I vote
we set them as the default github branches. Anyone opposed?

—
Reply to this email directly or view it on GitHub
#449 (comment)
.

[vrabaud]

thx, closing that in favor of #490

[davetcoleman]

moveit_core, moveit_ros, and moveit_planners now default to the indigo branch