search

moveit / srdfdom

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

ROS2/Tinyxml2 updates #52

Closed pbeeson closed 4 years ago

pbeeson commented 4 years ago

Updates needed for ros2 compile (tested under eloquent) and tinyxml2.

henningkayser commented 4 years ago

@pbeeson Thanks a lot for preparing this! I've created a PR for your feature branch with some changes that should fix CI warnings (https://github.com/jdlangs/srdfdom/pull/1). Please review/merge and then squash the commit history into meaningful chunks (i.e. code migration, update CMakeLists.txt/package.xml, etc...)

pbeeson commented 4 years ago

@henningkayser Done.

henningkayser commented 4 years ago

@pbeeson thanks for merging. It looks like urdfdom_py is still missing. I created a new ROS 2 port of that which should get merged first in order to get CI to succeed. We'll need to add a rosinstall for that.

henningkayser commented 4 years ago

@pbeeson thanks