Skip to content

Commit 8772641

Browse files
committed
check-distro-clang.sh: Some fixes
1. Add a trap for Ctrl-C so that the script immediately exits. 2. archlinux/base moved to archlinux/archlinux. 3. Add Debian's oldstable, which is sometimes cared about. 4. Remove ":latest" from the images, as it is implied. 5. Prefix all images with "docker.io/" because some distributions might not ship a containers.conf. Signed-off-by: Nathan Chancellor <nathan@kernel.org>
1 parent 5cd26fc commit 8772641

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

check-distro-clang.sh

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
#!/usr/bin/env bash
22

3+
trap 'exit' INT
4+
35
# Make sure we have the required binaries
46
for NAME in podman docker; do
57
command -v "${NAME}" &>/dev/null && BINARY=${NAME}
@@ -10,17 +12,18 @@ if [[ -z ${BINARY} ]]; then
1012
fi
1113

1214
DOCKER_DISTROS=(
13-
archlinux/base:latest
15+
archlinux/archlinux
16+
debian:oldstable-slim
1417
debian:stable-slim
1518
debian:testing-slim
1619
debian:unstable-slim
17-
fedora:latest
20+
fedora
1821
fedora:rawhide
19-
opensuse/leap:latest
20-
opensuse/tumbleweed:latest
22+
opensuse/leap
23+
opensuse/tumbleweed
2124
ubuntu:18.04
2225
ubuntu:20.04
23-
ubuntu:latest
26+
ubuntu
2427
ubuntu:rolling
2528
ubuntu:devel
2629
)
@@ -60,6 +63,7 @@ chmod +x "${RUN_SCRIPT}"
6063

6164
rm "${BASE}"/results.log
6265
for DISTRO in "${DOCKER_DISTROS[@]}"; do
66+
DISTRO=docker.io/${DISTRO}
6367
"${BINARY}" pull "${DISTRO}"
6468
"${BINARY}" run \
6569
--rm \

0 commit comments

Comments
 (0)