search

ros2 / rosbag2

Apache License 2.0
285 stars 251 forks source link

[jazzy] Address clang warnings (backport #1794) #1810

Closed mergify[bot] closed 2 months ago

mergify[bot] commented 2 months ago

There are some new warnings on clang https://ci.ros2.org/view/nightly/job/nightly_linux_clang_libcxx/1993/gcc/new/folder.1817012694/

This is an automatic backport of pull request #1794 done by Mergify.

MichaelOrlov commented 2 months ago

Pulls: ros2/rosbag2#1810 Gist: https://gist.githubusercontent.com/MichaelOrlov/ce2c7ed4b6bf20e272cdc8ad71755ffe/raw/31e44786249441f3c4e3d4d96cba77db5abfb4eb/ros2.repos BUILD args: --packages-above-and-dependencies rosbag2_transport TEST args: --packages-above rosbag2_transport ROS Distro: jazzy Job: ci_launcher ci_launcher ran: https://ci.ros2.org/job/ci_launcher/14575

MichaelOrlov commented 2 months ago

The redhat build finished with warnings no CONNEXTDDS_DIR nor NDDSHOME specified. However, this is unrelated to this PR. Merging then.