Skip to content

Commit 6238500

Browse files
ci: add Lyrical to the build & test matrix (#17)
1 parent eb74a40 commit 6238500

1 file changed

Lines changed: 8 additions & 7 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414
strategy:
1515
fail-fast: false
1616
matrix:
17-
ros_distro: [jazzy, kilted, rolling]
17+
ros_distro: [jazzy, kilted, rolling, lyrical]
1818
container:
1919
image: ros:${{ matrix.ros_distro }}-ros-base
2020
# Default Docker /dev/shm is 64MB; one domain registry mmap is ~37MB
@@ -60,13 +60,14 @@ jobs:
6060
ros-${{ matrix.ros_distro }}-test-msgs \
6161
ros-${{ matrix.ros_distro }}-rosidl-typesupport-cpp \
6262
ros-${{ matrix.ros_distro }}-rosidl-runtime-cpp
63-
# rosidl_buffer is a Rolling-only package (introduced after Jazzy/
64-
# Kilted were branched) and test_msgs' exported imported targets
65-
# reference it. CMake side: find_package(rosidl_buffer QUIET) in
66-
# CMakeLists handles the apt-missing case on older distros.
67-
if [ "${{ matrix.ros_distro }}" = "rolling" ]; then
63+
# rosidl_buffer was introduced after Jazzy/Kilted were branched and
64+
# test_msgs' exported imported targets reference it; Lyrical branched
65+
# from Rolling after it landed, so both ship it. CMake side:
66+
# find_package(rosidl_buffer QUIET) in CMakeLists handles the
67+
# apt-missing case on older distros.
68+
if [ "${{ matrix.ros_distro }}" = "rolling" ] || [ "${{ matrix.ros_distro }}" = "lyrical" ]; then
6869
apt-get install -y --no-install-recommends \
69-
ros-rolling-rosidl-buffer
70+
ros-${{ matrix.ros_distro }}-rosidl-buffer
7071
fi
7172
7273
- name: Build

0 commit comments

Comments
 (0)