search

ros2 / ros1_bridge

ROS 2 package that provides bidirectional communication between ROS 1 and ROS 2
Apache License 2.0
452 stars 288 forks source link

[backport foxy] support idl files on the ROS 2 side of the bridge #301

Closed wjwwood closed 3 years ago

wjwwood commented 3 years ago

Backport of https://github.com/ros2/ros1_bridge/pull/296 and a subsequent bug fix #298.

wjwwood commented 3 years ago

This technically breaks public API, but as discussed in this comment thread:

https://github.com/ros2/ros1_bridge/pull/292#issuecomment-729889386

We agreed that this was unlikely to be a public API in use outside of ros1_bridge and not worth the effort of a API compatible rework.

wjwwood commented 3 years ago

CI:

wjwwood commented 3 years ago

The compiler warning in the CI is from rmw_fastrtps_shared_cpp, so it's unrelated.