Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Enable service typesupport for C #117

Merged
merged 1 commit into from
Mar 24, 2016
Merged

Enable service typesupport for C #117

merged 1 commit into from
Mar 24, 2016

Conversation

jacquelinekay
Copy link
Contributor

Connects to ros2/rcl#27

depends on #120

@jacquelinekay jacquelinekay added the in progress Actively being worked on (Kanban column) label Feb 24, 2016
@jacquelinekay jacquelinekay self-assigned this Feb 24, 2016
@jacquelinekay jacquelinekay mentioned this pull request Feb 24, 2016
5 tasks
@jacquelinekay jacquelinekay force-pushed the c_services branch 4 times, most recently from 65f8fbb to f645b49 Compare March 2, 2016 22:29
@jacquelinekay
Copy link
Contributor Author

OSX seems to have problems with the generated C code included the C++ code. Looking into it.

http://ci.ros2.org/job/ci_osx/872/console#console-section-0

@@ -267,6 +269,7 @@ ament_target_dependencies(
${rosidl_generate_interfaces_TARGET}${_target_suffix}
"OpenSplice"
"rosidl_typesupport_opensplice_c")
#"${PROJECT_NAME}__rosidl_generator_c")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about this comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

@wjwwood
Copy link
Member

wjwwood commented Mar 24, 2016

Other than the comments it lgtm. Please remember to quash though (lots of small commits here).

@jacquelinekay jacquelinekay merged commit ad7111f into master Mar 24, 2016
@jacquelinekay jacquelinekay deleted the c_services branch March 24, 2016 23:54
@jacquelinekay jacquelinekay self-assigned this Mar 24, 2016
@jacquelinekay jacquelinekay removed the in progress Actively being worked on (Kanban column) label Mar 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants