Skip to content

Commit

Permalink
Fix output declaration in build-linux job, and output references in t…
Browse files Browse the repository at this point in the history
…est-* jobs (again).
  • Loading branch information
afinetooth committed Sep 30, 2024
1 parent 36a21df commit 214c767
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,8 +69,8 @@ jobs:
build-linux:
runs-on: ubuntu-latest
outputs:
coveralls_x86_64: ${{ steps.build.outputs.coveralls_x86_64 }}
coveralls_aarch64: ${{ steps.build.outputs.coveralls_aarch64 }}
coveralls_x86_64: ${{ steps.package.outputs.coveralls_x86_64 }}
coveralls_aarch64: ${{ steps.package.outputs.coveralls_aarch64 }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
steps:
Expand All @@ -97,6 +97,7 @@ jobs:
run: ls -la dist/

- name: Package linux binaries
id: package
run: |
make package
echo "coveralls_x86_64=dist/coveralls-linux-x86_64" >> $GITHUB_OUTPUT
Expand Down Expand Up @@ -131,6 +132,7 @@ jobs:
- name: Test x86_64 binary
run: |
binary=${{ needs.build-linux.outputs.coveralls_x86_64 }}
echo $binary
chmod +x $binary
$binary --version
Expand Down Expand Up @@ -161,6 +163,7 @@ jobs:
- name: Test aarch64 binary in QEMU
run: |
binary=${{ needs.build-linux.outputs.coveralls_aarch64 }}
echo $binary
chmod +x $binary
sudo apt-get update
sudo apt-get install -y qemu-user
Expand Down

0 comments on commit 214c767

Please sign in to comment.