-
Notifications
You must be signed in to change notification settings - Fork 119
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
migrate moveit_full_pr2 from ros-planning/moveit #98
Conversation
... to resolve a circular package dependency (moveit/moveit#1111)
ba6d2fa
to
c45caaa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But CI is failing for some reason. I can take a brief look.
Looks like the indigo-devel branch of this repo is failing for ages on Travis: https://travis-ci.org/ros-planning/moveit_pr2/branches. Does anybody actively maintain this branch? |
That could mean the failing CI may not matter much, at least in the ROS distro release infrastructure. Since this PR is a blocker for moveit/moveit#1111 and ultimately moveit/moveit#1000, I'm merging this PR (then fix the failing CI separately). |
* [Indigo] Relocate moveit_full_pr2 repo (following moveit/moveit_pr2#98). * [I] Remove temporarily. Not entirely sure why CI failed, but I assume the version mismatch b/w the packages in the repo that the `moveit_full_pr2` is moved to. This change removes `moveit_full_pr2` pkg from the entire ROS distro land, but it'll soon be added back when release for `moveit_pr2` is made. https://travis-ci.org/ros/rosdistro/jobs/445789675 ``` ERROR: test.test_build_caches.test_build_caches ---------------------------------------------------------------------- Traceback (most recent call last): File "/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages/nose/case.py", line 197, in runTest self.test(*self.arg) File "/home/travis/build/ros/rosdistro/test/test_build_caches.py", line 57, in test_build_caches raise RuntimeError('\n'.join(errors)) RuntimeError: indigo: missing package.xml file for package "moveit_full_pr2" -------------------- >> begin captured logging << -------------------- rosdistro: DEBUG: Load index from "file:///home/travis/build/ros/rosdistro/index.yaml" rosdistro: DEBUG: Load cache from "http://repositories.ros.org/rosdistro_cache/indigo-cache.yaml.gz" rosdistro: DEBUG: Load file from "file:///home/travis/build/ros/rosdistro/indigo/distribution.yaml" rosdistro: DEBUG: Dropping release package XML cache for moveit_full_pr2 ```
Migrate moveit_full_pr2 from moveit/moveit#1111 to resolve a circular package dependency.
moveit_full_pr2
depends on:moveit_pr2
->pr2_moveit_plugins
->moveit_*
Thus, this meta-package belongs here and not into the moveit repo.
Also cherry-pick this to kinetic-devel, replacing dependency to
moveit_full
in package.xml with actual packages?