From 9837dd429a0998341aaf70b377f17506a7e9f0d5 Mon Sep 17 00:00:00 2001 From: Viktor Szakats Date: Sun, 22 Jun 2025 22:44:24 +0200 Subject: [PATCH] GHA/windows: drop MSYS2 runtime downgrades No longer necessary after bumping the default runtime to a version fixing the previously experienced performance drop. Thanks to MSYS2/Cygwin teams for the help and fix. Follow-up to 9a26be1e6ad45eb6c46af1d7a5e0be273b14fe1b #17708 Follow-up to d4896d94f2e9530d47bf519c9d9b790720bf10a4 #16424 Closes #17710 --- .github/workflows/windows.yml | 30 ------------------------------ 1 file changed, 30 deletions(-) diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 5866d0c687..217fe5b3a8 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -367,16 +367,6 @@ jobs: /usr/bin/pacman --noconfirm --noprogressbar --sync --needed openssh /c/ProgramData/chocolatey/choco.exe install --yes --no-progress --limit-output --timeout 180 --force stunnel || true - - name: 'downgrade msys2-runtime' - if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' && matrix.sys != 'msys' }} - timeout-minutes: 2 - # Downgrade to a known good MSYS2 runtime version to avoid the performance regression - # causing runtests.pl to run at 2-3x reduced speed. - run: | - if [[ "$(uname -a)" = *' 3.5'* ]]; then - exec /usr/bin/pacman --noconfirm --noprogressbar --upgrade https://mirror.msys2.org/msys/x86_64/msys2-runtime-3.5.4-8-x86_64.pkg.tar.zst - fi - - name: 'run tests' if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' }} timeout-minutes: 10 @@ -559,16 +549,6 @@ jobs: /c/ProgramData/chocolatey/choco.exe install --yes --no-progress --limit-output --timeout 180 --force stunnel || true python3 -m pip --disable-pip-version-check --no-input --no-cache-dir install --progress-bar off --prefer-binary impacket - - name: 'downgrade msys2-runtime' - if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' }} - timeout-minutes: 2 - # Downgrade to a known good MSYS2 runtime version to avoid the performance regression - # causing runtests.pl to run at 2-3x reduced speed. - run: | - if [[ "$(uname -a)" = *' 3.5'* ]]; then - exec /usr/bin/pacman --noconfirm --noprogressbar --upgrade https://mirror.msys2.org/msys/x86_64/msys2-runtime-3.5.4-8-x86_64.pkg.tar.zst - fi - - name: 'run tests' if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' }} timeout-minutes: 10 @@ -1007,16 +987,6 @@ jobs: python3 -m pip --disable-pip-version-check --no-input --no-cache-dir install --progress-bar off --prefer-binary impacket fi - - name: 'downgrade msys2-runtime' - if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' }} - timeout-minutes: 2 - # Downgrade to a known good MSYS2 runtime version to avoid the performance regression - # causing runtests.pl to run at 2-3x reduced speed. - run: | - if [[ "$(uname -a)" = *' 3.5'* ]]; then - exec /usr/bin/pacman --noconfirm --noprogressbar --upgrade https://mirror.msys2.org/msys/x86_64/msys2-runtime-3.5.4-8-x86_64.pkg.tar.zst - fi - - name: 'run tests' if: ${{ matrix.tflags != 'skipall' && matrix.tflags != 'skiprun' }} timeout-minutes: 10 -- 2.47.2