diff --git a/.github/workflows/colcon.yaml b/.github/workflows/colcon.yaml index ea0e2b89f..e753651de 100644 --- a/.github/workflows/colcon.yaml +++ b/.github/workflows/colcon.yaml @@ -32,7 +32,7 @@ jobs: apt update rosdep update source /opt/ros/${{matrix.config.rosdistro}}/setup.bash - rosdep install --from-paths src --ignore-src -y --skip-keys "slam_toolbox turtlebot3_gazebo gazebo_ros_pkgs" + rosdep install --from-paths src --ignore-src -y --skip-keys "slam_toolbox turtlebot3_gazebo gazebo_ros_pkgs octomap_server" shell: bash - name: Colcon Build (Release) run: | diff --git a/grid_map_core/test/EigenPluginsTest.cpp b/grid_map_core/test/EigenPluginsTest.cpp index 4c902b667..e168c5910 100644 --- a/grid_map_core/test/EigenPluginsTest.cpp +++ b/grid_map_core/test/EigenPluginsTest.cpp @@ -14,6 +14,13 @@ #include "grid_map_core/grid_map_core.hpp" +// GCC 13 has false positive warnings around array-bounds. +// Suppress them until this is fixed in upstream gcc. See +// https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114758 for more details. +#if defined(__GNUC__) && !defined(__clang__) +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Warray-bounds" +#endif TEST(EigenMatrixBaseAddons, numberOfFinites) { diff --git a/grid_map_cv/include/grid_map_cv/GridMapCvConverter.hpp b/grid_map_cv/include/grid_map_cv/GridMapCvConverter.hpp index 2538b8258..233c28a0d 100644 --- a/grid_map_cv/include/grid_map_cv/GridMapCvConverter.hpp +++ b/grid_map_cv/include/grid_map_cv/GridMapCvConverter.hpp @@ -20,6 +20,14 @@ #include #include +// GCC 13 has false positive warnings around stringop-overflow. +// Suppress them until this is fixed in upstream gcc. See +// https://gcc.gnu.org/bugzilla/show_bug.cgi?id=114758 for more details. +#if defined(__GNUC__) && !defined(__clang__) +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wstringop-overflow" +#endif + namespace grid_map { diff --git a/grid_map_filters/CMakeLists.txt b/grid_map_filters/CMakeLists.txt index 9d0cba5a5..711302f25 100644 --- a/grid_map_filters/CMakeLists.txt +++ b/grid_map_filters/CMakeLists.txt @@ -136,6 +136,7 @@ if(BUILD_TESTING) list(APPEND AMENT_LINT_AUTO_EXCLUDE ament_cmake_cpplint ament_cmake_copyright + ament_cmake_uncrustify ) ament_lint_auto_find_test_dependencies() @@ -144,6 +145,17 @@ if(BUILD_TESTING) ament_cpplint( FILTERS -legal/copyright -build/include_order ) + + # run uncrustify except for EigenLab.hpp + find_package(ament_cmake_uncrustify) + set( + _linter_excludes + include/EigenLab/EigenLab.hpp + ) + ament_uncrustify( + EXCLUDE ${_linter_excludes} + LANGUAGE c++ + ) endif() ament_lint_auto_find_test_dependencies() endif() diff --git a/grid_map_ros/src/GridMapRosConverter.cpp b/grid_map_ros/src/GridMapRosConverter.cpp index 91ff6d8a3..23a6edc47 100644 --- a/grid_map_ros/src/GridMapRosConverter.cpp +++ b/grid_map_ros/src/GridMapRosConverter.cpp @@ -688,7 +688,7 @@ bool GridMapRosConverter::saveToBag( auto bag_message = std::make_shared(); - auto ret = rcutils_system_time_now(&bag_message->time_stamp); + auto ret = rcutils_system_time_now(&bag_message->send_timestamp); if (ret != RCL_RET_OK) { RCLCPP_ERROR(rclcpp::get_logger("saveToBag"), "couldn't assign time rosbag message"); } diff --git a/tools/ros2_dependencies.repos b/tools/ros2_dependencies.repos index 9a0dc8dd9..23d88992a 100644 --- a/tools/ros2_dependencies.repos +++ b/tools/ros2_dependencies.repos @@ -1,5 +1,6 @@ repositories: ros-planning/navigation2: type: git - url: https://github.com/ros-planning/navigation2.git - version: main + # back to upstream main after https://github.com/ros-navigation/navigation2/pull/4298 is merged + url: https://github.com/tonynajjar/navigation2.git + version: fix-devcontainer