Skip to content

Remove tinyxml_vendor.#1502

Merged
clalancette merged 1 commit intorollingfrom
clalancette/remove-tinyxml-vendor
Dec 27, 2023
Merged

Remove tinyxml_vendor.#1502
clalancette merged 1 commit intorollingfrom
clalancette/remove-tinyxml-vendor

Conversation

@clalancette
Copy link
Copy Markdown
Contributor

Now that urdfdom no longer requires it, we shouldn't need it at all.

Still a draft until I run CI and verify that it is truly not needed anymore.

Now that urdfdom no longer requires it, we shouldn't need
it at all.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette
Copy link
Copy Markdown
Contributor Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@clalancette clalancette linked an issue Dec 26, 2023 that may be closed by this pull request
3 tasks
@clalancette clalancette marked this pull request as ready for review December 27, 2023 13:18
@clalancette clalancette requested a review from codebot as a code owner December 27, 2023 13:18
@clalancette clalancette merged commit ed49952 into rolling Dec 27, 2023
@clalancette clalancette deleted the clalancette/remove-tinyxml-vendor branch December 27, 2023 13:18
Jiusi-pys pushed a commit to Jiusi-pys/ros2 that referenced this pull request Jan 17, 2026
Now that urdfdom no longer requires it, we shouldn't need
it at all.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TinyXML2 Migration Tracking Issue

2 participants