Skip to content
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

pr2_moveit_tutorials is missing a dependency on shape_tools #64

Closed
tfoote opened this issue Feb 5, 2016 · 8 comments
Closed

pr2_moveit_tutorials is missing a dependency on shape_tools #64

tfoote opened this issue Feb 5, 2016 · 8 comments

Comments

@tfoote
Copy link

tfoote commented Feb 5, 2016

It's the one regression in indigo at the moment. I suspect it was relying on a transitive dependency which was cleaned up so now it's broken.

This is blocking the build of moveit_pr2 and moveit_full_pr2

http://build.ros.org/view/Ibin_uS64/job/Ibin_uS64__pr2_moveit_tutorials__ubuntu_saucy_amd64__binary/10/console

CMakeFiles/pick_place_tutorial.dir/src/pick_place_tutorial.cpp.o -c /tmp/binarydeb/ros-indigo-pr2-moveit-tutorials-0.6.1/pick_place/src/pick_place_tutorial.cpp
00:12:36.857 /tmp/binarydeb/ros-indigo-pr2-moveit-tutorials-0.6.1/pick_place/src/pick_place_tutorial.cpp:42:46: fatal error: shape_tools/solid_primitive_dims.h: No such file or directory
00:12:36.857  #include <shape_tools/solid_primitive_dims.h>
00:12:36.857                     
@IanTheEngineer
Copy link
Contributor

Thanks for the report, Tully. I actually resolved this the other day:
#63
and cut a new release this morning:
https://github.com/ros-planning/moveit_pr2/releases/tag/0.6.2
Unfortunately, I seem to be lacking write permissions required to push to ros-gbp org (ros_planning team) for bloom.

@IanTheEngineer
Copy link
Contributor

One thing to note, we seem to have a version mismatch on moveit_commander and moveit_plugins between the most recent releases and those that were merged into rosdistro. @130s noticed this:

ros/rosdistro#10378 (comment)
moveit_commander: rosdistro says 0.5.7-1, sand should be 0.6.0-0
ros/rosdistro#10374 (comment)
moveit_plugins: rosdistro says 0.5.6-2, sand should be 0.5.7-0

We shouldn't do a sync until this discrepancy is rectified.

@tfoote
Copy link
Author

tfoote commented Feb 5, 2016

@IanTheEngineer I see you on the team now. Can you rerun bloom?

@IanTheEngineer
Copy link
Contributor

I can re-run it tonight on my home workstation. My work machine's environment is non-standard, as you noticed: http://answers.ros.org/question/223663/dependencies-not-satisfied-ros-indigo-gazebo5-msgs/

If anyone else in @ros-planning wants to rebloom those 3 packages in the meantime, I would not object

@mikeferguson
Copy link
Contributor

I can also help out --- after 4pm today

@IanTheEngineer
Copy link
Contributor

Resolved :) ros/rosdistro#10433

@cdrwolfe
Copy link

Anyone know what the actual fix was for this? Seems to be replicated on the turtlebot_arm package as well when building in catkin:

fatal error: shape_tools/solid_primitive_dims.h: No such file or directory
 #include <shape_tools/solid_primitive_dims.h>

https://github.com/corb555/turtlebot_arm

@130s
Copy link
Contributor

130s commented Feb 26, 2017

@cdrwolfe #63?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants