mrpt-ros-pkg / mrpt_slam

ROS wrappers for SLAM algorithms in MRPT
http://wiki.ros.org/mrpt_slam
BSD 3-Clause "New" or "Revised" License
114 stars 49 forks source link

Noetic mrpt_icp_slam_2d mrpt_rbpf_slam failing to build: ‘now’ is not a member of ‘mrpt::system’ #72

Closed sloretz closed 8 months ago

sloretz commented 8 months ago

Maybe these need a new release too?

23:07:39 /tmp/binarydeb/ros-noetic-mrpt-rbpf-slam-0.1.15/src/mrpt_rbpf_slam.cpp: In destructor ‘virtual mrpt_rbpf_slam::PFslam::~PFslam()’:
23:07:45 /tmp/binarydeb/ros-noetic-mrpt-rbpf-slam-0.1.15/src/mrpt_rbpf_slam.cpp:20:48: error: ‘now’ is not a member of ‘mrpt::system’
23:07:45    20 |   mrpt::system::timestampToParts(mrpt::system::now(), parts, true);
23:07:45       |                                                ^~~
23:07:45 make[4]: *** [CMakeFiles/mrpt_rbpf_slam.dir/build.make:66: CMakeFiles/mrpt_rbpf_slam.dir/src/mrpt_rbpf_slam.cpp.o] Error 1
23:07:36 /tmp/binarydeb/ros-noetic-mrpt-icp-slam-2d-0.1.15/src/mrpt_icp_slam_2d_wrapper.cpp: In destructor ‘ICPslamWrapper::~ICPslamWrapper()’:
23:07:48 /tmp/binarydeb/ros-noetic-mrpt-icp-slam-2d-0.1.15/src/mrpt_icp_slam_2d_wrapper.cpp:41:34: error: ‘now’ was not declared in this scope; did you mean ‘pow’?
23:07:48    41 |   mrpt::system::timestampToParts(now(), parts, true);
23:07:48       |                                  ^~~
23:07:48       |                                  pow
23:07:48 make[4]: *** [CMakeFiles/mrpt_icp_slam_2d.dir/build.make:66: CMakeFiles/mrpt_icp_slam_2d.dir/src/mrpt_icp_slam_2d_wrapper.cpp.o] Error 1
jlblancoc commented 8 months ago

Right!

Done here: https://github.com/ros/rosdistro/pull/40327