Skip to content

fix(autoware_cmake): remove workaround, fixed in https://github.com/ros2/tinyxml2_vendor/pull/23#30

Open
esteve wants to merge 1 commit intoautowarefoundation:mainfrom
esteve:remove-tinxyml2-workaround
Open

fix(autoware_cmake): remove workaround, fixed in https://github.com/ros2/tinyxml2_vendor/pull/23#30
esteve wants to merge 1 commit intoautowarefoundation:mainfrom
esteve:remove-tinxyml2-workaround

Conversation

@esteve
Copy link
Copy Markdown
Contributor

@esteve esteve commented May 22, 2025

Description

This PR removes the workaround for autowarefoundation/autoware_universe#10410

How was this PR tested?

This was tested on an Ubuntu 22.04 container, with and without the workaround.

Notes for reviewers

None.

Effects on system behavior

None.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@esteve
Copy link
Copy Markdown
Contributor Author

esteve commented May 22, 2025

Do not merge until ros2/tinyxml2_vendor#23 is merged and backported to Humble

@esteve esteve force-pushed the remove-tinxyml2-workaround branch from a677624 to 87ea7ec Compare May 22, 2025 12:57
@esteve esteve changed the title fix: remove workaround, fixed in https://github.com/ros2/tinyxml2_vendor/pull/23 fix(autoware_cmake): remove workaround, fixed in https://github.com/ros2/tinyxml2_vendor/pull/23 May 22, 2025


Signed-off-by: Esteve Fernandez <esteve.fernandez@tier4.jp>
@esteve esteve force-pushed the remove-tinxyml2-workaround branch from 87ea7ec to b0f1f4f Compare November 25, 2025 16:29
@esteve
Copy link
Copy Markdown
Contributor Author

esteve commented Nov 25, 2025

@mitsudome-r @youtalk ros2/tinyxml2_vendor#27 and ros2/tinyxml2_vendor#28 are now merged and released, this PR can now be merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants