ros-drivers / transport_drivers

A set of ROS2 drivers for transport-layer protocols.
Apache License 2.0
92 stars 56 forks source link

New release breaks builds in Autoware.Auto #62

Closed niosus closed 2 years ago

niosus commented 3 years ago

Seems like the new release is causing the pipelines to fail. See example: https://gitlab.com/autowarefoundation/autoware.auto/AutowareAuto/-/jobs/1561321563

I can already see that the udp_driver_node.hpp has been removed. Do you guys think it warrants to have a list of breaking changes for the release? Ideally, also a transition guide.

JWhitleyWork commented 3 years ago

There were only 2 organizations (that we know of) which were actively using them and this release goes from version 0.0.6 to 1.0.0 - breaking changes were intended. The paradigm of how you use the classes in these has completely changed so a list of "changes" isn't really feasible. Previously you had to inherit from one of the node-based classes that were included in these packages but you can now use them as stand-alone C++ classes, as shown in the example nodes.

niosus commented 3 years ago

@JWhitleyWork could you point me to the examples you are referring to here? Are you talking about the tests?

JWhitleyWork commented 3 years ago

Specifically https://github.com/ros-drivers/transport_drivers/blob/main/serial_driver/src/serial_bridge_node.cpp, https://github.com/ros-drivers/transport_drivers/blob/main/udp_driver/src/udp_receiver_node.cpp, and https://github.com/ros-drivers/transport_drivers/blob/main/udp_driver/src/udp_sender_node.cpp.

JWhitleyWork commented 2 years ago

This was resolved in https://gitlab.com/autowarefoundation/autoware.auto/AutowareAuto/-/merge_requests/1170.