Skip to content

#177: Update Connext to 7.3.0#181

Merged
nuclearsandwich merged 4 commits intorollingfrom
freature/177-update-connext-730
Apr 8, 2025
Merged

#177: Update Connext to 7.3.0#181
nuclearsandwich merged 4 commits intorollingfrom
freature/177-update-connext-730

Conversation

@lobolanja
Copy link
Collaborator

@lobolanja lobolanja commented Apr 1, 2025

This PR intends to update the rmw_connextmodule to use RTI Connext DDS 7.3.0 as explained in #177

we will need also the work on:

ROS CI PR: ros2/ci#811
rosdistro PR: ros/rosdistro#44918

@lobolanja lobolanja changed the title #177: change depend to 7.3.0, TODO documentatation. #177: Update Connext to 7.3.0 Apr 1, 2025
@bijoua29
Copy link

bijoua29 commented Apr 5, 2025

I hope this PR will be approved soon and make it to Kilted as I note that rmw freeze is coming up on Apr 7. We are dependent on using Connext 7.3.0 as our non-ROS applications will be using 7.3.0.

@lobolanja
Copy link
Collaborator Author

@mjcarroll @nuclearsandwich
Could you do a quick review of the changes in rti_connext_dds_cmake_module/package.xml?

I added the -ros suffix since that matches the package we provided last week and aligns with how it’s expected to be retrieved from the OSRF Debian repositories. I believe this should also be consistent with the changes in ros/rosdistro#44918.

Thanks in advance!

Copy link
Member

@nuclearsandwich nuclearsandwich left a comment

Choose a reason for hiding this comment

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

None of the documentation or metadata changes in this PR are actually exercised by ROS 2 CI. CI for the move to connext 7.3 is being run in ros2/ci#811 (comment)

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.

5 participants