xacro doesn't properly parse ROS substitution_args

[smits]

See ticket on code.ros.org: https://code.ros.org/trac/ros-pkg/ticket/5420

[piyushk]

Just tested groovy_devel, and the patch seems to be working well.

[jonbinney]

Here's where this stands now:

• substitution args patch has been merged into hydro-devel and released into hydro
• its been merged into groovy-devel, but the groovy-devel version on github is catkinized, and there's too much risk that the catkinization will break come use cases, so we can't release again. Some work needs to be done to roll back the groovy-devel version here to rosbuild, and then re-apply the patch. Not sure I have time to do this right now, but if somebody does, I'll release it.
• there is a separate issue for this in fuerte: Backport xacro substitution args fix to fuerte? #5

[jonbinney]

Went ahead and rolled the groovy-devel branch back to the last rosbuild version, and then backported the substitution args fix. The tests pass and script seems to work, but I'd like to get a couple other people to try it out and make sure I haven't broken anything before I do a release.

[wjwwood]

Closing, the catkinized version is out in groovy and hydro. Please open a new ticket if this is still an open issue.