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

[RePR from #114] Fixed missing C interfaces to obtain service and action type support. #143

Merged
merged 1 commit into from May 15, 2023

Conversation

StefanFabian
Copy link
Contributor

Currently, only message type support contains a C interface to dynamically load the type support from a library.
This PR adds a C interface for services and actions.

New PR since the previous fork was unfortunately deleted and could not be rebased. For discussions see #114 .

Signed-off-by: Stefan Fabian <github@stefanfabian.com>
@clalancette
Copy link
Contributor

CI for this is in ros2/rosidl#703 (comment)

@clalancette clalancette merged commit 9062d91 into ros2:rolling May 15, 2023
3 checks passed
@ros-discourse
Copy link

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/preparing-for-foxy-fitzroy-end-of-life/31470/2

@achim-k
Copy link

achim-k commented Nov 23, 2023

@Mergifyio backport humble

(Probably won't work for me, can one of the contributors trigger that?)

Copy link

mergify bot commented Nov 23, 2023

backport humble

❌ Command disallowed due to command restrictions in the Mergify configuration.

  • sender-permission>=write

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

5 participants