Skip to content

Ignore rosdep key for new RTI Connext DDS 7.3.0 version#962

Merged
christophebedard merged 1 commit intomasterfrom
christophebedard/connext-7
Apr 15, 2025
Merged

Ignore rosdep key for new RTI Connext DDS 7.3.0 version#962
christophebedard merged 1 commit intomasterfrom
christophebedard/connext-7

Conversation

@christophebedard
Copy link
Copy Markdown
Member

@christophebedard christophebedard commented Apr 14, 2025

ROS 2 Rolling switched to version 7.3.0 (before Kilted): ros2/rmw_connextdds#181.

This PR is similar to ros2/ros2_documentation#5227. Just ignore the rosdep key for RTI Connext DDS 7.3.0.

Relates to this setup-ros PR: ros-tooling/setup-ros#798.

Relates a bit to ros2/ros2_documentation#5226

Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard self-assigned this Apr 14, 2025
@christophebedard christophebedard requested a review from a team as a code owner April 14, 2025 21:37
@christophebedard christophebedard requested review from MichaelOrlov and emersonknapp and removed request for a team April 14, 2025 21:37
@christophebedard christophebedard merged commit a3c6c01 into master Apr 15, 2025
32 of 36 checks passed
@christophebedard christophebedard deleted the christophebedard/connext-7 branch April 15, 2025 00:07
Copy link
Copy Markdown
Member

@MichaelOrlov MichaelOrlov left a comment

Choose a reason for hiding this comment

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

@christophebedard Thanks for fixing this. I've just yestarday came across with the issue of the new RTI Connext DDS when were trying ti build a docker image for ros-ade.
I am curious if we will have the same issue with Jazzy or not?

@christophebedard
Copy link
Copy Markdown
Member Author

Not with Jazzy, because rmw_connextdds was bumped to Connext 7.3.0 only in Rolling: ros2/rmw_connextdds#181

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.

2 participants