Skip to content

Commit

Permalink
Fix windows CI PG13 failure
Browse files Browse the repository at this point in the history
Installing the postgres13 windows package pulled in an install
of vcredist140 which requires a reboot making chocolatey return
with a non-zero exit code to signal the required reboot. This
patch changes chocolatey to no longer give individual packages
control over chocolatey exit code. This patch also changes
chocolatey to longer show package download progress information
to make the CI log less spammy.
  • Loading branch information
svenklemm committed Apr 17, 2021
1 parent d26c744 commit d34af2a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/windows-build-and-test.yaml
Expand Up @@ -44,7 +44,10 @@ jobs:
key: ${{ runner.os }}-build-pg${{ matrix.pkg_version }}
- name: Install PostgreSQL ${{ matrix.pg }}
if: steps.cache-postgresql.outputs.cache-hit != 'true'
run: choco install postgresql${{ matrix.pg }} --version ${{ matrix.pkg_version }} --force -y --install-args="'--prefix $HOME\PostgreSQL\${{ matrix.pg }} --extract-only yes'"
run: |
choco feature disable --name=usePackageExitCodes
choco feature disable --name=showDownloadProgress
choco install postgresql${{ matrix.pg }} --version ${{ matrix.pkg_version }} --force -y --install-args="'--prefix $HOME\PostgreSQL\${{ matrix.pg }} --extract-only yes'"
- name: Configure [${{ matrix.build_type }}]
run: cmake -B ${{ matrix.build_type }} -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -DREGRESS_CHECKS=OFF -DPG_PATH="$HOME/PostgreSQL/${{ matrix.pg }}" -DOPENSSL_ROOT_DIR="$HOME/PostgreSQL/${{ matrix.pg }}"
# Build step: could potentially speed things up with --parallel
Expand Down

0 comments on commit d34af2a

Please sign in to comment.