search

ros-drivers / openni2_camera

ROS wrapper for openni 2.0
http://wiki.ros.org/openni2_camera
BSD 3-Clause "New" or "Revised" License
56 stars 96 forks source link

Build fails on Ubuntu Noble (24.04) #137

Closed mikeferguson closed 6 months ago

mikeferguson commented 6 months ago

https://build.ros2.org/job/Rbin_uN64__openni2_camera__ubuntu_noble_amd64__binary/4/console

03:07:31                  from /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/src/openni2_convert.cpp:32:
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/include/openni2_camera/openni2_device_info.h:45:3: error: ‘uint16_t’ does not name a type
03:07:31    45 |   uint16_t vendor_id_;
03:07:31       |   ^~~~~~~~
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/include/openni2_camera/openni2_device_info.h:36:1: note: ‘uint16_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
03:07:31    35 | #include <ostream>
03:07:31   +++ |+#include <cstdint>
03:07:31    36 | 
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/include/openni2_camera/openni2_device_info.h:46:3: error: ‘uint16_t’ does not name a type
03:07:31    46 |   uint16_t product_id_;
03:07:31       |   ^~~~~~~~
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/include/openni2_camera/openni2_device_info.h:46:3: note: ‘uint16_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/src/openni2_convert.cpp: In function ‘const openni2_wrapper::OpenNI2DeviceInfo openni2_wrapper::openni2_convert(const openni::DeviceInfo*)’:
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/src/openni2_convert.cpp:50:10: error: ‘struct openni2_wrapper::OpenNI2DeviceInfo’ has no member named ‘product_id_’
03:07:31    50 |   output.product_id_ = pInfo->getUsbProductId();
03:07:31       |          ^~~~~~~~~~~
03:07:31 /tmp/binarydeb/ros-rolling-openni2-camera-2.1.0/src/openni2_convert.cpp:51:10: error: ‘struct openni2_wrapper::OpenNI2DeviceInfo’ has no member named ‘vendor_id_’; did you mean ‘vendor_’?
03:07:31    51 |   output.vendor_id_  = pInfo->getUsbVendorId();

This is going to be easiest to fix once the first rolling sync happens - then our CI will be useful again

mikeferguson commented 6 months ago

CI is passing after merging #138