From: Viktor Szakats Date: Wed, 19 Mar 2025 21:13:40 +0000 (+0100) Subject: GHA/linux: fix filter expressions for skipall/skiprun X-Git-Tag: curl-8_13_0~109 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7d313e603b42c7640a4dea8327cd69289c3c6eac;p=thirdparty%2Fcurl.git GHA/linux: fix filter expressions for skipall/skiprun For cases when `install_steps` contains extra components. After this patch, msh3 and rustls CM jobs skip building and running tests, saving 2 minutes CI time, as originally intended. Closes #16772 --- diff --git a/.github/workflows/linux.yml b/.github/workflows/linux.yml index 8182b8ae4f..1f3c5a1b7e 100644 --- a/.github/workflows/linux.yml +++ b/.github/workflows/linux.yml @@ -303,7 +303,7 @@ jobs: sudo apt-get -o Dpkg::Use-Pty=0 update sudo apt-get -o Dpkg::Use-Pty=0 install \ libtool autoconf automake pkgconf ninja-build \ - ${{ matrix.build.install_steps != 'skipall' && matrix.build.install_steps != 'skiprun' && 'stunnel4' || '' }} \ + ${{ !contains(matrix.build.install_steps, 'skipall') && !contains(matrix.build.install_steps, 'skiprun') && 'stunnel4' || '' }} \ libpsl-dev libbrotli-dev libzstd-dev \ ${{ matrix.build.install_packages }} \ ${{ contains(matrix.build.install_steps, 'pytest') && 'apache2 apache2-dev libnghttp2-dev vsftpd' || '' }} @@ -643,7 +643,7 @@ jobs: run: cmake --install bld --strip - name: 'build tests' - if: ${{ matrix.build.install_steps != 'skipall' }} + if: ${{ !contains(matrix.build.install_steps, 'skipall') }} run: | if [ -n '${{ matrix.build.generate }}' ]; then cmake --build bld --verbose --target testdeps @@ -652,13 +652,13 @@ jobs: fi - name: 'install test prereqs' - if: ${{ matrix.build.install_steps != 'skipall' && matrix.build.container == null }} + if: ${{ !contains(matrix.build.install_steps, 'skipall') && matrix.build.container == null }} run: | [ -x "$HOME/venv/bin/activate" ] && source $HOME/venv/bin/activate python3 -m pip install -r tests/requirements.txt - name: 'run tests' - if: ${{ matrix.build.install_steps != 'skipall' && matrix.build.install_steps != 'skiprun' }} + if: ${{ !contains(matrix.build.install_steps, 'skipall') && !contains(matrix.build.install_steps, 'skiprun') }} timeout-minutes: ${{ contains(matrix.build.install_packages, 'valgrind') && 30 || 15 }} run: | export TFLAGS='${{ matrix.build.tflags }}'