Skip to content

Removed deprecated geometry_msgs/Pose2d#283

Merged
ahcorde merged 1 commit intorollingfrom
ahcorde/rolling/removed_deprecated_pose2d
Jun 17, 2025
Merged

Removed deprecated geometry_msgs/Pose2d#283
ahcorde merged 1 commit intorollingfrom
ahcorde/rolling/removed_deprecated_pose2d

Conversation

@ahcorde
Copy link
Copy Markdown
Contributor

@ahcorde ahcorde commented May 22, 2025

Pose2D.msg is deprecated since Foxy.

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested review from mjcarroll and tfoote May 22, 2025 08:23
@ahcorde ahcorde self-assigned this May 22, 2025
@ahcorde
Copy link
Copy Markdown
Contributor Author

ahcorde commented Jun 4, 2025

Pulls: #283
Gist: https://gist.githubusercontent.com/ahcorde/f3b4d95ea5261bf2c159ca6c86d01d8f/raw/d76c5f01a2d1d9447d16744d1a3a468e8161f08f/ros2.repos
BUILD args: --packages-above-and-dependencies geometry_msgs
TEST args: --packages-above geometry_msgs
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16167

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

@ahcorde
Copy link
Copy Markdown
Contributor Author

ahcorde commented Jun 4, 2025

@tfoote what do you think ? can we remove this message ?

@ahcorde ahcorde merged commit da15991 into rolling Jun 17, 2025
3 checks passed
@ahcorde ahcorde deleted the ahcorde/rolling/removed_deprecated_pose2d branch June 17, 2025 09:37
@DLu
Copy link
Copy Markdown
Contributor

DLu commented Aug 21, 2025

I'm very late to the party, but this is a bad idea. It has very "Google kills Google Reader" energy.

@tfoote
Copy link
Copy Markdown
Contributor

tfoote commented Sep 2, 2025

It's good to clean things up here, we don't do it a lot. We did discuss our processes for doing this with better communications in the ROS PMC meetings with the above cross referenced breakages.

I think it would be valuable to point to there are reasonable alternatives such as

https://github.com/ros/ros_tutorials/blob/rolling/turtlesim_msgs/msg/Pose.msg

It might be good to add that in as a comment to the release notes @ahcorde with guidance on how to quickly migrate.

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.

4 participants