28 | @@ -29,6 +29,10 @@ if [ -n "${APT_LLVM_V}" ]; then
29 | # shellcheck disable=SC2034
30 | source /etc/os-release
31 | echo "deb http://apt.llvm.org/${VERSION_CODENAME}/ llvm-toolchain-${VERSION_CODENAME}-${APT_LLVM_V} main" > "/etc/apt/sources.list.d/llvm-toolchain-${VERSION_CODENAME}-${APT_LLVM_V}.list"
32 | + # Temporarily work around Sequoia PGP policy deadline for legacy repositories.
33 | + # See https://github.com/llvm/llvm-project/issues/153385.
34 | + sed -i 's/\(sha1\.second_preimage_resistance =\).*/\1 9999-01-01/' /usr/share/apt/default-sequoia.config && \
35 | + apt-get update && \
I don't like the && \ here. I think we should globally exit on non-zero exit code, but i guess this is needed in a sub-shell?
My preference would be to use set -o errexit -o pipefail -o xtrace