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
Fix get type support handle #9
Conversation
Like the custom headers provided by the connext and opensplice implementations, the rosidl_typesupport_introspection_cpp package provides a generic interface for getting the type support handle. This is used by the rmw_connext_xtypes_dynamic_cpp package. This also fixes linking on Windows.
+1 |
1 similar comment
+1 |
@@ -62,6 +62,8 @@ add_library(${rosidl_generate_interfaces_TARGET}${_target_suffix} SHARED ${_gene | |||
if(WIN32) | |||
target_compile_definitions(${rosidl_generate_interfaces_TARGET}${_target_suffix} | |||
PRIVATE "ROSIDL_TSI_CPP_BUILDING_DLL") | |||
target_compile_definitions(${rosidl_generate_interfaces_TARGET}${_target_suffix} | |||
PRIVATE "ROSIDL_BUILDING_DLL") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the CMake docs it looks like that you can specify both definitions in a single invocation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I actually meant to remove the flag above, along with the whole visibility header in the rosidl_typesupport_introspection_cpp
package. I'll update it one way or another.
This pull request adds an implementation of the
get_type_support_handle
functions for therosidl_typesupport_introspection_cpp
package. This used by thermw_connext_xtypes_dynamic_cpp
package to get a type support handle.Connects to ros2/ros2#26