SteveMacenski / slam_toolbox

Slam Toolbox for lifelong mapping and localization in potentially massive maps with ROS
GNU Lesser General Public License v2.1
1.67k stars 525 forks source link

Unable to use exported libraries due to relative imports in header #471

Closed dimasad closed 2 years ago

dimasad commented 2 years ago

Required Info:

Steps to reproduce issue

Expected behavior

The toolbox types can be used in another package.

Actual behavior

It fails to compile because the imports in the headers are relative, so only slam_toolbox can be compiled. There's a comment that says "god... getting this to work in ROS2 was a real pain", so I believe that it the fix broke inter-package compatibility.

Additional information

SteveMacenski commented 2 years ago

PRs have been merged - closing