Skip to content

Commit

Permalink
fix installbuilder not able to produce complete artifacts for linux/w…
Browse files Browse the repository at this point in the history
…in (#855)

The condition was wrong: we were only using the else branch.
This was causing the linux installers to not contain the actual binary.
We do not support running installbuilder with `linux` as a target, we support `linux-x64`
  • Loading branch information
umbynos authored Nov 16, 2023
1 parent 3ba51ce commit 13515f7
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -370,14 +370,17 @@ jobs:
include:
- os: ubuntu-20.04
platform-name: linux
installbuilder-name: linux-x64
installer-extension: .run
- os: windows-2019
arch: 386
platform-name: windows
installbuilder-name: windows
extension: .exe
installer-extension: .exe
- os: windows-2019
platform-name: windows
installbuilder-name: windows
extension: .exe
installer-extension: .exe

Expand Down Expand Up @@ -418,12 +421,7 @@ jobs:

# installbuilder reads the env vars with certs paths and use it to sign the installer.
- name: Launch Bitrock installbuilder
run: |
if [[ ${{matrix.platform-name}} == "linux" ]]; then
${{ env.INSTALLBUILDER_PATH }} build installer.xml ${{ matrix.platform-name }}-x64 --verbose --license /tmp/license.xml --setvars ${{ env.INSTALLER_VARS }} architecture=${{ matrix.arch }}
else
${{ env.INSTALLBUILDER_PATH }} build installer.xml ${{ matrix.platform-name }} --verbose --license /tmp/license.xml --setvars ${{ env.INSTALLER_VARS }} architecture=${{ matrix.arch }}
fi
run: ${{ env.INSTALLBUILDER_PATH }} build installer.xml ${{ matrix.installbuilder-name }} --verbose --license /tmp/license.xml --setvars ${{ env.INSTALLER_VARS }} architecture=${{ matrix.arch }}

- name: Generate archive
run: tar -czvf ArduinoCreateAgent-${GITHUB_REF##*/}-${{ matrix.platform-name }}-${{ matrix.arch }}-installer.tar.gz ArduinoCreateAgent-${GITHUB_REF##*/}-${{ matrix.platform-name }}-${{ matrix.arch }}-installer${{matrix.installer-extension}}
Expand Down

0 comments on commit 13515f7

Please sign in to comment.