Skip to content

[APG-2626] orbbec_camera: port to ROS L (lyrical) - #30

Open
bmagyar wants to merge 8 commits into
locusfrom
ci/lyrical-ament-fix
Open

[APG-2626] orbbec_camera: port to ROS L (lyrical)#30
bmagyar wants to merge 8 commits into
locusfrom
ci/lyrical-ament-fix

Conversation

@bmagyar

@bmagyar bmagyar commented Jul 2, 2026

Copy link
Copy Markdown
Member

Ports orbbec_camera to build on ROS L (lyrical / Ubuntu 26.04), kept jazzy-compatible throughout. Validated with a colcon build on lyrical.

  • Replace ament_target_dependencies (removed in lyrical) with modern target_link_libraries (pkg::pkg / ${pkg_TARGETS}); drop image_publisher from link targets and declare message_filters explicitly.
  • Migrate .h.hpp headers for tf2, tf2_ros, cv_bridge, message_filters (present on both jazzy and lyrical).
  • Guard the message_filters Subscriber QoS API (removed rmw_qos_profile_t overload → rclcpp::QoS) on RCLCPP_VERSION_GTE(32,0,0) so both distros compile.
  • -Wno-error=deprecated-declarations: lyrical deprecates the remaining rmw_qos_profile_t QoS overloads this driver uses (image_transport, rclcpp); keep them warnings, not errors. (Full QoS modernization left as a follow-up.)
  • Drop an unused image_publisher/image_publisher.hpp include.

@bmagyar bmagyar changed the title [APG-2626] orbbec_camera: replace ament_target_dependencies with modern target_link_libraries [APG-2626] orbbec_camera: port to ROS L (lyrical) Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant