-
Notifications
You must be signed in to change notification settings - Fork 230
Fix response future type #198
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
Conversation
Any idea why CI didn't catch this @tylerjw ? |
No idea. I didn't look into why CI didn't catch this but I did reproduce it locally. |
This also happened in the scheduled build of foxy 2 days ago: https://github.com/ros-planning/moveit2_tutorials/runs/4126809816?check_suite_focus=true. So needs to be backported to foxy. |
Seems foxy and galactic is broken, rolling is fine with this included or not. I say we merge this in all distros. |
I tested this on rolling too. |
(cherry picked from commit 9710af8)
(cherry picked from commit 9710af8)
Fixes: "warning: ‘rclcpp::Client<ServiceT>::FutureAndRequestId::operator rclcpp::Client<ServiceT>::SharedFuture() [with ServiceT = moveit_msgs::srv::ApplyPlanningScene; rclcpp::Client<ServiceT>::SharedFuture = std::shared_future<std::shared_ptr<moveit_msgs::srv::ApplyPlanningScene_Response_<std::allocator<void> > > >; typename ServiceT::Response::SharedPtr = std::shared_ptr<moveit_msgs::srv::ApplyPlanningScene_Response_<std::allocator<void> > >]’ is deprecated: FutureAndRequestId: use .future.share() instead of an implicit conversion [-Wdeprecated-declarations]"
Description
I've tested this fixing this error on galactic. It needs to be also tested on rolling:
Checklist