Skip to content

Commit

Permalink
git-artifacts: fix BUILD_ONLY handling for ARM64
Browse files Browse the repository at this point in the history
The workflow allows users to restrict what parts are being built. For
example, `installer-i686` will build only the 32-bit installer, not the
64-bit one nor any MinGit flavor.

However, this logic was not extended when introducing support for ARM64:
Instead, we _also_ built the ARM64 installer when the user asked for
`installer-i686`.

Let's allow restricting to `installer-i686` _without_ building the ARM64
version, and allow restricting to `installer-arm64` _just_ for the ARM64
version.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
  • Loading branch information
dscho committed Sep 22, 2022
1 parent 60208ad commit cefb9bf
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/workflows/git-artifacts.yml
Expand Up @@ -338,9 +338,14 @@ jobs:
- name: Determine whether this job should be skipped
shell: bash
run: |
suffix=${{matrix.arch.name}}
if test true = ${{matrix.arch.arm64}}
then
suffix=arm64
fi
case " $BUILD_ONLY " in
' ') ;; # not set; build all
*" ${{matrix.artifact.name}} "*|*" ${{matrix.artifact.name}}-${{matrix.arch.name}} "*) ;; # build this artifact
*" ${{matrix.artifact.name}} "*|*" ${{matrix.artifact.name}}-$suffix "*) ;; # build this artifact
*) echo "SKIP=true" >>$GITHUB_ENV;;
esac
- name: Download pkg-${{matrix.arch.name}}
Expand Down

0 comments on commit cefb9bf

Please sign in to comment.