Skip to content

Add support for rmw_connextdds#182

Merged
clalancette merged 2 commits intoros2:masterfrom
asorbini:asorbini/rmw_connextdds
Mar 24, 2021
Merged

Add support for rmw_connextdds#182
clalancette merged 2 commits intoros2:masterfrom
asorbini:asorbini/rmw_connextdds

Conversation

@asorbini
Copy link
Copy Markdown
Contributor

@asorbini asorbini commented Mar 5, 2021

This PR replaces all references to rmw_connext_cpp with rmw_connextdds.

See rticommunity/rmw_connextdds #9 for a list of related PRs, and an overview of all the changes required to replace ros2/rmw_connext (rmw_connext_cpp) with rticommunity/rmw_connextdds in the ROS2 source tree.

Signed-off-by: Andrea Sorbini <asorbini@rti.com>
…xtdds

Signed-off-by: Andrea Sorbini <asorbini@rti.com>
@asorbini asorbini changed the title Replace rmw_connext_cpp with rmw_connextdds Add support for rmw_connextdds Mar 11, 2021
@ivanpauno
Copy link
Copy Markdown
Member

CI, just in case:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@ivanpauno
Copy link
Copy Markdown
Member

the Rpr checker will pass when rmw_connextdds gets released for rolling

@ivanpauno
Copy link
Copy Markdown
Member

ivanpauno commented Mar 11, 2021

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link
Copy Markdown
Contributor

the Rpr checker will pass when rmw_connextdds gets released for rolling

Right, we have to wait to merge this one until we have the new package bloomed and building in Rolling.

@clalancette
Copy link
Copy Markdown
Contributor

@ros-pull-request-builder retest this please

@clalancette
Copy link
Copy Markdown
Contributor

All right, CI over in ros2/rmw_connextdds#9 (comment) was green, so I'll go ahead and merge this.

@clalancette clalancette merged commit 864a444 into ros2:master Mar 24, 2021
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