Skip to content

Commit 75ab96f

Browse files
ci(release): Use container in workflow instead of build-in-docker.sh
This eases maintainability, as the workflow is direcltly defined in `build.yml`. This will also allow us to add caching of dependencies more easily, and make the workflow for Linux more closely match the workflow for macOS and Windows.
1 parent bffbead commit 75ab96f

File tree

2 files changed

+6
-39
lines changed

2 files changed

+6
-39
lines changed

.github/workflows/build.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,17 @@ jobs:
2727
name: Linux ${{ matrix.arch }}
2828
runs-on: ubuntu-24.04
2929

30+
container:
31+
image: messense/rust-musl-cross:${{ matrix.container }}
32+
3033
steps:
3134
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # 5.0.0
3235

33-
- name: Build in Docker
34-
run: scripts/build-in-docker.sh
35-
env:
36-
TARGET: ${{ matrix.target }}
37-
DOCKER_TAG: ${{ matrix.container }}
36+
- name: Build
37+
run: cargo build --release --target=${{ matrix.target }} --locked
3838

3939
- name: Rename Binary
40-
run: mv target/*/release/sentry-cli sentry-cli-Linux-${{ matrix.arch }}
40+
run: mv target/${{ matrix.target }}/release/sentry-cli sentry-cli-Linux-${{ matrix.arch }}
4141

4242
- uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # 4.6.2
4343
with:

scripts/build-in-docker.sh

Lines changed: 0 additions & 33 deletions
This file was deleted.

0 commit comments

Comments
 (0)