Backport xacro substitution args fix to fuerte?

[jbohren]

#4 fixes broken usage of substitution_args, this is also trivially fixed in the fuerte version, if I submit a PR, can that get released into fuerte?

[ablasdel]

At the moment xacro doesn't build off of github due to a problem with the catkinization and related debacle.
Once #2 is reliably fixed and we move to building off of github we can look into backports.

[jbohren]

What's the status of this?

[ablasdel]

@jonbinney Is that fix you were discussing before I left in/working now? If so what are your thoughts on a backport? crazy or only half crazy?

[jonbinney]

The hydro-devel branch has the workaround for #2 and has been released into hydro. The groovy devel branch also has the fix, but has not been released. Both have the substitution args fix.

I don't think this really matters for this specific issue though. Backporting the substitution args fix to fuerte would involve releasing from github, but not catkinizing.

[jonbinney]

@jbohren - I created a (currently empty) fuerte-devel branch in this repo. If you merge the substitution args code into the old xacro code and pull request it to fuerte-devel, i'll do the release.

[jonbinney]

I went ahead and backported the fix into the fuerte-devel branch of this repo. @jbohren @ablasdel can you confirm that this looks OK and works reasonably on fuerte for you?

[wjwwood]

I'm going to close this as stale, since we are not building fuerte debs anymore.

[jbohren]

Sounds reasonable.