Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Support for Connext 6 #375

Closed
dirk-thomas opened this issue Oct 16, 2019 · 9 comments
Closed

Support for Connext 6 #375

dirk-thomas opened this issue Oct 16, 2019 · 9 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@dirk-thomas
Copy link
Member

Some preliminary work regarding Connext 6 support has landed in #371, ros2/rosidl_typesupport_connext#37, and #374 to allow adding Connext 6 support at any time without breaking anything. The patch file connext_static_serialized_dataPlugin.cxx.patch which contains the heavy lifting and still needs to be updated for Connext 6 though.

@Karsten1987 @neil-rti I will leave this up to either of you (or anyone else) to contribute the missing bits in a new pull request. Until then ROS 2 will be limited to only support Connext 5.

With the conditional logic in place such a change can land any time - it just won't change the Connext version we are building against in already released distributions though - since that is specified in REP 2000 and locked before a release.

@dirk-thomas dirk-thomas added enhancement New feature or request help wanted Extra attention is needed labels Oct 16, 2019
@Karsten1987
Copy link
Contributor

I am very much in favor of replacing the custom logic with a more elegant and future prone solution. However, I currently don't have an easy way of doing so and I am currently not able to spend more time on this, I'd kindly ask @neil-rti or somebody with more insights in the Connext 6.0 API to help out.

@AndreasAZiegler
Copy link

May I ask about the current status of Connext 6 support?

@ivanpauno
Copy link
Member

I'm starting with #381.
A lot of collisions are possible between the tasks, so I recommend waiting for #381 before starting with this one (less rebasing work).

AFAIK, there is nobody working on this. If that's not the case, let me know ASAP.

@AndreasAZiegler
Copy link

Are there any update?

@dirk-thomas
Copy link
Member Author

dirk-thomas commented Mar 4, 2020

Are there any update?

No comment implies nothing has changed since the last comments - see above:

  • "I'd kindly ask @neil-rti or somebody with more insights in the Connext 6.0 API to help out."
  • "AFAIK, there is nobody working on this."

@neil-rti
Copy link
Contributor

neil-rti commented Mar 6, 2020

It's a work in process; ideally we'd like to fix the build issues and address the performance bottleneck in the RMW layer for both 5.3 and 6.0 versions. I can't give a firm ETA at this point.

@dirk-thomas
Copy link
Member Author

@neil-rti Do you think it is realistic to expect that this will happen and get merged for Foxy (which means before the freeze deadline on April 8th)?

@neil-rti
Copy link
Contributor

neil-rti commented Mar 9, 2020

@dirk-thomas Probably not, although we may have contributions to merge for the 5.3.1 RMW layer.

@clalancette
Copy link
Contributor

Closing, since with Foxy now being End-of-Life, this repository is no longer used.

@clalancette clalancette closed this as not planned Won't fix, can't repro, duplicate, stale Jun 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants