ros-industrial / ros_industrial_issues

Repository for tracking common ROS-Industrial issuses.
3 stars 4 forks source link

Migrate the package.xml files to format 2 #56

Closed ipa-nhg closed 4 years ago

ipa-nhg commented 5 years ago

(or make more sense wait and migrate to format 3 directly http://www.ros.org/reps/rep-0149.html ?)

Lists of packages to be migrated:

gavanderhoorn commented 5 years ago

I think first getting everything to version 2 would be better. Changes to get to 3 are minimal, especially with the packages that we have: we don't have dependency groups, don't release into ROS2 (yet) and the ABI spec is nice, but not something of immediate concern.

gavanderhoorn commented 5 years ago

@ipa-nhg: could you perhaps update the issue with a list of pkgs that need to be updated?

ipa-nhg commented 5 years ago

Lists of packages to be migrated:

gavanderhoorn commented 5 years ago

@ipa-nhg: thanks for the list.

For future reference: what command(s) did you use to find these packages?

ipa-nhg commented 5 years ago

I create a .rosinstall file with all the ros-industrial repos which contains ros packages, cloned from the rosinstall file and commanded: grep -R "<package>"

I think will be useful push somewhere the rosinstall file (the industrial_metapackages repo could be an option or just create a new one industrial_basics? industrial_helper? industrial_helper_tools? ), also we can work on some basic bash script to check things like this.

gavanderhoorn commented 5 years ago

For anyone that wants to work on this during wrid19: please coordinate and announce the repository/packages that you're going to work on just so we can avoid duplication of effort.

dejaniraai commented 5 years ago

I will start with the industrial_core packages

dejaniraai commented 5 years ago

I will start with the industrial_core packages

I have made a PR for this package. I will move this issue back to the unassigned so that someone can deal with the other packages.

fmauch commented 5 years ago

I'll start working on universal_robot

Fern-and-dough commented 5 years ago

I'll start working on abb_experimental

dhled commented 5 years ago

@ipa-nhg robotiq packages have through re-factoring. All package.xml have been updated to version 2 for kinetic devel.

gavanderhoorn commented 5 years ago

@ipa-led: I've updated the OP.

Fern-and-dough commented 5 years ago

Working on Staubli as well

gavanderhoorn commented 5 years ago

I've updated the OP to show the current state of things and unassigned the issue itself.

Thanks @wahlent, @fmauch, @dejaniraai, @Fern-and-dough and @trentwall for working on this and contributing your PRs! :+1:

hsd-dev commented 4 years ago

https://github.com/ros-industrial/stomp_ros already has format 2 for the melodic-devel branch

hsd-dev commented 4 years ago

https://github.com/ros-industrial/industrial_moveit/pull/102
https://github.com/ros-industrial/motoman_experimental/pull/47
https://github.com/ros-industrial/staubli_experimental/pull/40

ipa-nhg commented 4 years ago

seems to be completed, can this issue be closed?

gavanderhoorn commented 4 years ago

If all referenced PRs have been merged, then yes, it would seem this can be closed.

Thanks for the reminder.