Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fixing CI] using chrono literals for durations from rclcpp API update #69

Merged
merged 3 commits into from
Dec 7, 2020

Conversation

SteveMacenski
Copy link
Member

Per https://app.circleci.com/pipelines/github/ros-planning/navigation2/4121/workflows/80503470-8d6a-4e4f-bbee-65a65e1cab04/jobs/15663

/opt/ros/foxy/include/rclcpp/duration.hpp:46:12: note: declared here
   46 |   explicit Duration(rcl_duration_value_t nanoseconds);
      |            ^~~~~~~~
In file included from /opt/overlay_ws/src/navigation2/nav2_util/include/nav2_util/lifecycle_node.hpp:25,
                 from /opt/overlay_ws/src/navigation2/nav2_util/src/lifecycle_node.cpp:15:
/opt/underlay_ws/install/bondcpp/include/bondcpp/bond.hpp:142:76: error: ‘rclcpp::Duration::Duration(rcl_duration_value_t)’ is deprecated: Use Duration::from_nanoseconds instead or std::chrono_literals. For example:rclcpp::Duration::from_nanoseconds(int64_variable);rclcpp::Duration(0ns); [-Werror=deprecated-declarations]
  142 |   bool waitUntilBroken(rclcpp::Duration timeout = rclcpp::Duration(-1 * 1e9));
      |                                                                            ^

Bondcpp with ROS2 main is a little unhappy with the use of duration integers without from_nanoseconds (new) or chrono literals. I updated everything to use chrono literals.

Using this job as a test https://app.circleci.com/jobs/github/ros-planning/navigation2/15671 that this removes the deprecation warning

@SteveMacenski
Copy link
Member Author

Also, I'd be more than happy to be added as a ROS2 maintainer for bond since its in my dependency tree for Nav2. That way I can fix these types of things up more quickly. At least until the new ROS2 bond replacement using DDS features is complete / ready.

@SteveMacenski
Copy link
Member Author

Job shows that its no longer bondcpp failing - this resolves the issue. The failures remaining are having to fix the same deprecation across the nav2 stack itself.

@SteveMacenski
Copy link
Member Author

15:29:19 CMake Error at /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:146 (message):
15:29:19   Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
15:29:19 Call Stack (most recent call first):
15:29:19   /usr/share/cmake-3.16/Modules/FindPackageHandleStandardArgs.cmake:393 (_FPHSA_FAILURE_MESSAGE)
15:29:19   /usr/share/cmake-3.16/Modules/FindPkgConfig.cmake:41 (find_package_handle_standard_args)
15:29:19   CMakeLists.txt:19 (find_package)

This rolling PR builder is failing from unrelated things

@mjcarroll
Copy link
Member

Interesting, I'll take a look at this today.

@SteveMacenski
Copy link
Member Author

@mjcarroll any thoughts?

@mjcarroll
Copy link
Member

Sorry, this plus #70 should at least get us building (but not passing tests)

@mjcarroll mjcarroll merged commit 433b70d into ros:ros2 Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants