Skip to content

Commit 05a5150

Browse files
ci(release): Use container in workflow instead of build-in-docker.sh (#2727)
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. We also remove the smoke test, which never triggered anyways, as it was relying on Travis environment variables.
1 parent 479dc78 commit 05a5150

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)