ros2 / ci

ROS 2 CI Infrastructure
http://ci.ros2.org/
Apache License 2.0
48 stars 30 forks source link

Install the liborocos-kdl-dev system package #668

Closed cottsay closed 2 years ago

cottsay commented 2 years ago

This package is already installed for RHEL.

Linux Build Status

cottsay commented 2 years ago

Should we hold off on merging this...?

@nuclearsandwich and @Blast545, does a Docker cache bust worry you right now?

Blast545 commented 2 years ago

@cottsay I haven't seen the error in any of the nightlies, I think it's good to merge now. I'm re running your original build to make double check before merging.

Linux CI: Build Status

Blast545 commented 2 years ago

LGTM to merge ping

nuclearsandwich commented 2 years ago

@nuclearsandwich and @Blast545, does a Docker cache bust worry you right now?

Bring it on. If problems are created that means that #667 is incomplete somehow and needs to be dealt with.