search

moveit / srdfdom

Semantic Robot Description Format
BSD 3-Clause "New" or "Revised" License
12 stars 68 forks source link

Use new urdf SharedPtr typedefs #17

Closed nxdefiant closed 8 years ago

nxdefiant commented 8 years ago

Urdf has moved to a new SharedPtr type which results in a compile error:

/opt/ros/ros_catkin_ws/src/srdfdom/src/model.cpp: In member function 'void srdf::Model::loadGroups(const urdf::ModelInterface&, TiXmlElement*)':
/opt/ros/ros_catkin_ws/src/srdfdom/src/model.cpp:182:43: error: variable 'boost::shared_ptr<const urdf::Link> l' has initializer but incomplete type
       boost::shared_ptr<const urdf::Link> l = urdf_model.getLink(tip_str);

Replacing boost::shared_ptr<const urdf::Link> l with urdf::LinkConstSharedPtr l

fixes this error.

rhaschke commented 8 years ago

Addressed in #18. Note, that urdfdom 1.0 (that has the transition to std::shared_ptr) is not released yet in any Ubuntu release. Xenial/Kinetic still comes with version 0.4.1.

nxdefiant commented 8 years ago

But Debian testing/unstable already has it. #18 compiles, thanks.