IntelRealSense / librealsense2-release

release repository for ROS distribution
Apache License 2.0
15 stars 10 forks source link

Include realsense tools (again) #6

Open stwirth opened 2 years ago

stwirth commented 2 years ago

In ROS kinetic releases, the very useful realsense tools such as rs-enumerate-devices and rs-fw-update were included, see e.g.

https://github.com/IntelRealSense/librealsense2-release/blob/release/kinetic/librealsense2/2.45.0-1/CMake/lrs_options.cmake#L15

In ROS noetic releases, this option seems to have been off for a few versions now.

Since https://github.com/IntelRealSense/librealsense/pull/8802, building of TOOLS and EXAMPLES is split. I would propose to set BUILD_TOOLS to ON for ROS packages as these tools are essential for working with robots.

@doronhi

doronhi commented 2 years ago

I agree. I will put a note on to add the tools in the next ros-librealsense2 release. I am not sure though when is the next release scheduled.

stwirth commented 2 years ago

Great, thank you!