Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for rmw_connextdds #182

Merged
merged 2 commits into from
Mar 24, 2021

Conversation

asorbini
Copy link
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
Member

CI, just in case:

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

@ivanpauno
Copy link
Member

the Rpr checker will pass when rmw_connextdds gets released for rolling

@ivanpauno
Copy link
Member

ivanpauno commented Mar 11, 2021

CI:

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

@clalancette
Copy link
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
Contributor

@ros-pull-request-builder retest this please

@clalancette
Copy link
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.

None yet

4 participants