Skip to content

feat: add support for ROS 2 Jazzy#355

Merged
rsasaki0109 merged 4 commits intoMapIV:autoware-mainfrom
mitsudome-r:support-jazzy
Jan 22, 2026
Merged

feat: add support for ROS 2 Jazzy#355
rsasaki0109 merged 4 commits intoMapIV:autoware-mainfrom
mitsudome-r:support-jazzy

Conversation

@mitsudome-r
Copy link

@mitsudome-r mitsudome-r commented Jan 22, 2026

Description

This PR fixes all the build errors in ROS 2 Jazzy while keeping compatibility to ROS 2 Humble.
The fix includes:

  • missing dependency to ament_index_cpp in package.xml
  • update linking of geographiclib (similar fix with feat: support ROS 2 Jazzy llh_converter#25)
  • replacing deprecated headers for tf2_eigen/tf2_eigen.h and tf2_sensor_msgs/tf2_sensor_msgs.h (I also removed #ifdef for Galactic since Galactic has already reached EOL)
  • fix linking to yaml-cpp libraries

I can split the PR into each items if that makes easier to review.

How the PR was tested

I have confirmed that it builds fine in both Ubuntu 22.04(ROS 2 Humble) and Ubuntu 24.04 (ROS 2 Jazzy)

Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
…f headers

Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
Copy link

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

@rsasaki0109
Copy link
Collaborator

I also confirmed that it builds locally on ROS 2 Jazzy. After reviewing the code, everything looks good to me, so I'll merge it.

@rsasaki0109 rsasaki0109 merged commit 575136e into MapIV:autoware-main Jan 22, 2026
1 check passed
@xmfcx
Copy link

xmfcx commented Jan 23, 2026

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.

3 participants