search

ros-controls / ros_control

Generic and simple controls framework for ROS
http://wiki.ros.org/ros_control
BSD 3-Clause "New" or "Revised" License
470 stars 307 forks source link

Add xmlns for URDF, remove xmlns:xacro #436

Closed matthew-reynolds closed 4 years ago

matthew-reynolds commented 4 years ago

Drops the unused xmlns:xacro, since we aren't using any xacro, and replaces it with the URDF ns.

Although not widely used (As far as I can see), technically URDF should specify an xmlns of http://www.ros.org, as specified by the targetNamespace of the URDF schema. Might as well set a good example :)

Interesting note, this removes every instance of the string "xacro" in the repo.

bmagyar commented 4 years ago

Github-Travis integration is flaky again, build is fine: https://travis-ci.org/github/ros-controls/ros_control/builds/672780015