Skip to content

Commit fb0d1a8

Browse files
committed
fix(ci): build-cli-binaries matrix – use-cross, target add, cross install
- use-cross as string "true" so Install cross step runs (fixes cross: command not found) - Add explicit rustup target add for non-cross jobs (fixes target not installed) - Wrap conditionals in ${{ }} for reliable GHA evaluation
1 parent 6649144 commit fb0d1a8

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

.github/workflows/build-cli-binaries.yml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,14 +26,14 @@ jobs:
2626
- target: aarch64-unknown-linux-gnu
2727
os: ubuntu-latest
2828
package: cli-node-linux-arm64
29-
use-cross: true
29+
use-cross: "true"
3030
- target: x86_64-unknown-linux-musl
3131
os: ubuntu-latest
3232
package: cli-node-linux-x64-musl
3333
- target: aarch64-unknown-linux-musl
3434
os: ubuntu-latest
3535
package: cli-node-linux-arm64-musl
36-
use-cross: true
36+
use-cross: "true"
3737

3838
runs-on: ${{ matrix.os }}
3939
steps:
@@ -44,8 +44,12 @@ jobs:
4444
with:
4545
targets: ${{ matrix.target }}
4646

47-
- name: Install cross (for ARM targets)
48-
if: matrix.use-cross == 'true'
47+
- name: Add Rust target
48+
if: ${{ matrix.use-cross != 'true' }}
49+
run: rustup target add ${{ matrix.target }}
50+
51+
- name: Install cross (for ARM Linux targets)
52+
if: ${{ matrix.use-cross == 'true' }}
4953
run: cargo install cross --git https://github.com/cross-rs/cross
5054

5155
- name: Install musl-tools (for musl x64)

0 commit comments

Comments
 (0)