feat: Switch to c++20 and remove resulting compile warnings#3124
feat: Switch to c++20 and remove resulting compile warnings#3124skyegalaxy merged 1 commit intoros2:rollingfrom
Conversation
Signed-off-by: Janosch Machowinski <J.Machowinski@cellumation.com>
|
Pulls: ros2/ament_cmake_ros#62, ros2/rcutils#548, #3124 |
fujitatomoya
left a comment
There was a problem hiding this comment.
lgtm with green CI
|
Pulls: #3124, ros2/ament_cmake_ros#62, ros2/rcutils#548 |
|
Mhm... Looking at the log even with the parameter to 2022 it's not picking up MSVC2022. This seems like a bug on ci.ros2. I'll fix it and re-run. |
|
Are you sure? Logs say that it opens up this dev environment: |
|
You are correct, sorry for the noise. It's setting to MSVC2022 I saw on the zenoh output and got confused with Windows version numbers. |
skyegalaxy
left a comment
There was a problem hiding this comment.
per client library WG meeting today: we'll run CI one more time and if linux is green, we'll merge this, ros2/ament_cmake_ros#62 and ros2/rcutils#548
|
Pulls: #3124, ros2/ament_cmake_ros#62, ros2/rcutils#548 |
| <buildtool_depend>python3-empy</buildtool_depend> | ||
|
|
||
| <build_depend>ament_index_cpp</build_depend> | ||
| <build_depend>ament_index_cpp_core</build_depend> |
There was a problem hiding this comment.
I am seeing:
/usr/bin/rosdep:6: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
from pkg_resources import load_entry_point
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
rclcpp: Cannot locate rosdep definition for [ament_index_cpp_core]
after this PR, I think this should be ament_cmake_ros_core right?
I created a PR here: #3133
Description
Switch to cpp20 from lyrical onwards
Is this user-facing behavior change?
yes
Did you use Generative AI?
no