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

cleanup urdfdom compatibility #27

Merged
merged 2 commits into from
Jan 30, 2017

Conversation

rhaschke
Copy link
Contributor

... now relying on compatibility layer of urdf package: ros/robot_model#160

... now relying on compatibility layer of urdf package
@v4hn
Copy link
Contributor

v4hn commented Oct 28, 2016

Should we hold off this change until the robot_model update is synced?
Otherwise would this require changes to the buildfarm again so that tests of other requests still succeed?

@davetcoleman
Copy link
Member

+1 to holding off until sync

@v4hn
Copy link
Contributor

v4hn commented Jan 30, 2017

This is overdue. Merging.

@v4hn v4hn merged commit 84b056a into moveit:kinetic-devel Jan 30, 2017
130s pushed a commit to 130s/srdfdom that referenced this pull request Jan 31, 2017
130s pushed a commit to 130s/srdfdom that referenced this pull request Jan 31, 2017
130s pushed a commit to 130s/srdfdom that referenced this pull request Jan 31, 2017
130s pushed a commit to 130s/srdfdom that referenced this pull request Feb 3, 2017
130s added a commit to 130s/srdfdom that referenced this pull request Feb 5, 2017
v4hn added a commit that referenced this pull request Feb 6, 2017
[Indigo] cleanup urdfdom compatibility (cherry-picking #27)
@@ -41,8 +41,7 @@
#include <string>
#include <vector>
#include <utility>
#include <urdf_model/model.h>
#include <srdfdom/urdfdom_compatibility.h>
#include <urdf/model.h> // TODO: replace with urdf_model/types.h in Lunar
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

// TODO: replace with urdf_model/types.h in Lunar

Does this still hold?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried replacing it on a Ubuntu Zesty system and got an error:

/root/ros/ws_catkin/src/srdfdom/include/srdfdom/model.h:66:28: error: 'ModelInterface' in namespace 'urdf' does not name a type

Which I have not looked into

@rhaschke rhaschke deleted the urdfdom-compatibility branch March 22, 2018 08:12
@davetcoleman davetcoleman mentioned this pull request Mar 1, 2019
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

Successfully merging this pull request may close these issues.

4 participants