Closed sloretz closed 2 years ago
@hidmic I added you as a reviewer, but this one seems to depend on https://github.com/ros2/rosidl/pull/662 so I've left it as a draft until that one is merged.
CI failure involves rosidl_generator_py. I'll look into why it failed with this PR:
15:23:26 LINK : fatal error LNK1181: cannot open input file 'Release\rosidl_generator_py_custom__rosidl_typesupport_c.lib' [C:\ci\ws\build\rosidl_generator_py\rosidl_generator_py_custom__rosidl_generator_py.vcxproj]
CI LGTM :tada: The windows warning is in rmw_fastrtps
and was fixed by https://github.com/ros2/rmw_fastrtps/pull/592
This cleans up the rosidl generator code in the packages
rosidl_typesupport_c
and rosidl_typesupport_cpprosidl_typesupport_c
onlyrosidl_typesupport_cpp
onlyrcutils
rosidl_typesupport_c
androsidl_typesupport_cpp
ament_target_dependencies
withtarget_link_libraries()
ament/ament_cmake#292DEFINE_SYMBOL
for setting the compiler definition that toggles the visibility export macros.rosidl_generator_c
orrosidl_generator_cpp
, instead of hard coding include directories that they're going to createament_export_dependencies()
the packages needed by generated targets_generator_sources
check that can't work because other CMake code would error if it were ever actually emptydepends on ros2/rosidl#666Merged :tada:depend on ros2/rosidl#662Merged :tada: