appliedAI-Initiative / orb_slam_2_ros

A ROS implementation of ORB_SLAM2
Other
594 stars 283 forks source link

Add dependency definition for 'tf2_geometry_msgs' and 'tf2_ros' #116

Closed caioaamaral closed 2 years ago

caioaamaral commented 3 years ago

Description

Just tried manually build this package and noticed that some deps where missing, causing a fail at build phase. The main problem seems like tf2_geometry_msgs being missing, making the build crash.

But I also noticed that there are headers including tf2_ros stuff, but no dependency is defined on it. We could get it transitively from tf2_geometry_msgs, but the proper way would also add a dependency definition for tf2_ros

Overview

Add 'tf2_geometry_msgs' and 'tf2_ros' to package manifest and also find_package them at CMakeLists