Skip to content

Commit e6f17a0

Browse files
committed
fix(release): use if/else for protoc install, add apt-get update for aarch64
The || chain causes both dnf and yum to run on x86_64 manylinux (AlmaLinux 8 has yum as alias for dnf) — both complete, then apt-get is tried and fails with 'command not found'. The aarch64 rust-cross container is Debian-based and requires apt-get update before install (stale package list = 404 errors). Use if/else: dnf for manylinux x86_64, apt-get for aarch64 cross. Closes #55
1 parent ce6a673 commit e6f17a0

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

.github/workflows/release.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,9 +142,11 @@ jobs:
142142
manylinux: ${{ matrix.manylinux || 'auto' }}
143143
target: ${{ matrix.target }}
144144
before-script-linux: |
145-
command -v dnf && dnf install -y protobuf-compiler ||
146-
command -v yum && yum install -y protobuf-compiler ||
147-
command -v apt-get && apt-get install -y -q protobuf-compiler
145+
if command -v dnf >/dev/null 2>&1; then
146+
dnf install -y protobuf-compiler
147+
else
148+
apt-get update -q && apt-get install -y -q protobuf-compiler
149+
fi
148150
149151
- name: Upload wheels
150152
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4

0 commit comments

Comments
 (0)