Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

fix doTransform() for Eigen::Quaternion#496

Closed
gleichdick wants to merge 4 commits intoros:noetic-develfrom
gleichdick:fix_doTransform
Closed

fix doTransform() for Eigen::Quaternion#496
gleichdick wants to merge 4 commits intoros:noetic-develfrom
gleichdick:fix_doTransform

Conversation

@gleichdick
Copy link

Backport of ros2/geometry2#369

ooeygui pushed a commit to ms-iot/geometry2 that referenced this pull request Oct 12, 2022
* Export a tf2_geometry_msgs::tf2_geometry_msgs target

Signed-off-by: Shane Loretz <sloretz@openrobotics.org>

* Comment that tf2_ros is used but intentionally not linked

Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
Copy link
Contributor

@peci1 peci1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this was merged into ROS 2, is there anything blocking this PR?

It seems to me tf2_ros is a test-only dependency, but it was added as <depend>. Could you change it to <test_depend> only, please? (and do according changes in CMakeLists)

@ahcorde ahcorde added the noetic label Feb 2, 2024
@sloretz
Copy link
Contributor

sloretz commented Apr 23, 2025

Thank you for the PR!

ROS Noetic will reach end-of-life on May 31st, 2025. Every change comes with a risk of introducing regressions, and there isn't much time left to fix them. To make sure this PR doesn't introduce any regressions please:

  • Describe how you tested this change
  • Recruit at least one more person to review this PR and try it out on their system

@sloretz
Copy link
Contributor

sloretz commented May 31, 2025

ROS 1 is end-of-life (EOL) as of today, May 31st 2025. I am archiving this repository because:

  • it only supports ROS 1
  • it isn't needed anymore in ROS 2

If you still rely on ROS 1, read this page to learn about your options.

@sloretz sloretz closed this May 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants