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

rmw_connextdds: 0.1.1-1 in 'rolling/distribution.yaml' [non-bloom] #28559

Closed
wants to merge 1 commit into from

Conversation

asorbini
Copy link
Contributor

Introduce rticommunity/rmw_connextdds (0.1.1-1) to rolling.

This PR is linked to work being done to phase out ros2/rmw_connext (see rticommunity/rmw_connextdds#9).

The following packages will be added:

  • rmw_connextdds
  • rmw_connextddsmicro
  • rmw_connextdds_common
  • rti_connext_dds_cmake_module

The packages depend on RTI Connext DDS 5.3.1+.
Package rmw_connextddsmicro will not actually include an RMW implementation, since the library generation is disabled if RTI Connext Micro is not available (which will be the case).

The release repository currently points to rticommunity/rmw_connextdds-release, only because the name of the repository in the ros2-gbp org needs to be updated with the -release suffix. Once the name is fixed, I will push the existing commits to the new repository, or re-run bloom with the new repository. @clalancette (or anyone with access) can you update the repository name?

I'm not an expert on this release process (first time going through it), so I welcome any input.

While reviewing the packages to file this PR, I noticed that rti_connext_dds_cmake_module is missing a build dependency from rti-connext-dds-5.3.1, so I'll probably have to create a new release (which I can do with the release repository in the ros2-gbp org).

Signed-off-by: Andrea Sorbini <asorbini@rti.com>
@clalancette
Copy link
Contributor

The release repository currently points to rticommunity/rmw_connextdds-release, only because the name of the repository in the ros2-gbp org needs to be updated with the -release suffix. Once the name is fixed, I will push the existing commits to the new repository, or re-run bloom with the new repository. @clalancette (or anyone with access) can you update the repository name?

Very sorry, totally my mistake. I've now renamed the repository to https://github.com/ros2-gbp/rmw_connextdds-release , so you should be able to do a release to there. I'll close this PR out, as bloom will generate a new one when you release it.

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