NVIDIA-ISAAC-ROS / isaac_ros_common

Common utilities, packages, scripts, Dockerfiles, and testing infrastructure for Isaac ROS packages.
https://developer.nvidia.com/isaac-ros-gems
Other
197 stars 140 forks source link

Add ament_cmake_auto as build dep #36

Closed FelipeGdM closed 2 years ago

FelipeGdM commented 2 years ago

Hi!

This fixes a small issue found when trying to compile the project from source alongside ROS 2 core packages. The package ament_cmake_auto is required inside isaac_ros_apriltag_interfaces CMakeLists.txt, but it is not listed as build dependency in the package.xml. This way, colcon may try to build isaac_ros_apriltag_interfaces before ament_cmake_auto was built, resulting in an error.

This PR adds ament_cmake_auto as build dependency, thus informing colcon the correct order to build the packages