Add ros-ROS1_DISTRO-xmlrpcpp install

ros2 / ci

ROS 2 CI Infrastructure

http://ci.ros2.org/

Apache License 2.0

48 stars 30 forks source link

Add ros-ROS1_DISTRO-xmlrpcpp install #669

Closed methylDragon closed 2 years ago

methylDragon commented 2 years ago

The merge of https://github.com/ros2/ros1_bridge/pull/331 adds a new dependency to ros1_bridge: xmlrpcpp. I've seen some packaging jobs that pass despite not having this, but I think it's safer to include it?

So this just adds a rule to install the ros-${ROS1_DISTRO}-xmlrpcpp package, which exists across a ton of ROS1 distributions.

methylDragon commented 2 years ago

Example:

Without this branch:
With this branch:

methylDragon commented 2 years ago

Oh no..

methylDragon commented 2 years ago

Okay, I think jammy isn't supposed to work anyway, I'll close this (for now)