From b2ba1a489df451cdcd93b2334e319dd06778de19 Mon Sep 17 00:00:00 2001 From: Jia Tan Date: Wed, 18 Jan 2023 22:11:05 +0800 Subject: [PATCH] CI: Reorder 32-bit build first for Linux autotool builds. The 32-bit build needs to be first so the configure cache only needs to be reset one time. The 32-bit build sets the CFLAGS env variable, so any build using that flag after will fail unless the cache is reset. --- .github/workflows/ci.yml | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index bbf24d81..25716873 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -35,16 +35,23 @@ jobs: # -b specifies the build system to use. # -p specifies the phase (build or test) to help narrow down an error # if one occurs. + # + # Start with the 32-bit build because the autoconf cache must be reset + # after the build because the 32-bit build sets the CFLAGS env variable. + # By starting with the 32-bit build, we only have to clear the + # cache once. + - name: Build 32-bit + run: ./build-aux/ci_build.sh -b autotools -p build -f "-m32" + - name: Test 32-bit + run: | + ./build-aux/ci_build.sh -b autotools -p test -f "-m32" + cd ../xz_build && make distclean + - name: Build with full features run: ./build-aux/ci_build.sh -b autotools -p build - name: Test with full features run: ./build-aux/ci_build.sh -b autotools -p test - - name: Build 32-bit - run: ./build-aux/ci_build.sh -b autotools -p build -f "-m32" - - name: Test 32-bit - run: ./build-aux/ci_build.sh -b autotools -p test -f "-m32" - - name: Build without encoders run: ./build-aux/ci_build.sh -b autotools -d encoders,shared -p build - name: Test without encoders