PR2 / pr2_common

45 stars 79 forks source link

Prepend xacro tags with xacro xml namespace #259

Closed davetcoleman closed 7 years ago

davetcoleman commented 7 years ago

When using in Kinetic xacro outputs the following warning:

deprecated: xacro tags should be prepended with 'xacro' xml namespace.
Use the following script to fix incorrect usage:
        find . -iname "*.xacro" | xargs sed -i 's#<\([/]\?\)\(if\|unless\|include\|arg\|property\|macro\|insert_block\)#<\1xacro:\2#g'

so i did that.

archielee commented 7 years ago

ping @TheDash

davetcoleman commented 7 years ago

@v4hn do you know who is maintaining the PR2 repos now?

hvpandya commented 7 years ago

Would be great to have #259 and #260 merged

shupx commented 3 years ago

When using in Kinetic xacro outputs the following warning:

deprecated: xacro tags should be prepended with 'xacro' xml namespace.
Use the following script to fix incorrect usage:
        find . -iname "*.xacro" | xargs sed -i 's#<\([/]\?\)\(if\|unless\|include\|arg\|property\|macro\|insert_block\)#<\1xacro:\2#g'

so i did that.

excellent! This helps me a lot! it is necessary to fix this for namespace of xacro.