Skip to content

Commit d97f1e0

Browse files
committed
Also exclude spirv-tools-cli from pure Rust CI jobs
spirv-tools-cli depends on spirv-tools-ffi, so excluding only spirv-tools-ffi is not sufficient - cargo still tries to build it as a transitive dependency.
1 parent d5f8c31 commit d97f1e0

1 file changed

Lines changed: 9 additions & 6 deletions

File tree

.github/workflows/rust.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,13 +48,15 @@ jobs:
4848
4949
- name: Build Rust workspace
5050
working-directory: rust
51-
# Exclude spirv-tools-ffi which requires C++ build (tested in rust-ffi-tests job)
52-
run: cargo build --workspace --exclude spirv-tools-ffi
51+
# Exclude crates requiring C++ build (tested in rust-ffi-tests job)
52+
# spirv-tools-cli depends on spirv-tools-ffi, so both must be excluded
53+
run: cargo build --workspace --exclude spirv-tools-ffi --exclude spirv-tools-cli
5354

5455
- name: Run Rust tests
5556
working-directory: rust
56-
# Exclude spirv-tools-ffi which requires C++ build (tested in rust-ffi-tests job)
57-
run: cargo test --workspace --exclude spirv-tools-ffi
57+
# Exclude crates requiring C++ build (tested in rust-ffi-tests job)
58+
# spirv-tools-cli depends on spirv-tools-ffi, so both must be excluded
59+
run: cargo test --workspace --exclude spirv-tools-ffi --exclude spirv-tools-cli
5860

5961
# Rust FFI pretending to be C++ - runs C++ test suite against Rust implementation
6062
rust-ffi-tests:
@@ -156,6 +158,7 @@ jobs:
156158

157159
- name: Run Clippy
158160
working-directory: rust
159-
# Exclude spirv-tools-ffi which requires C++ build (tested in rust-ffi-tests job)
161+
# Exclude crates requiring C++ build (tested in rust-ffi-tests job)
162+
# spirv-tools-cli depends on spirv-tools-ffi, so both must be excluded
160163
# Lint configuration is in rust/Cargo.toml [workspace.lints.clippy]
161-
run: cargo clippy --workspace --exclude spirv-tools-ffi --all-targets -- -D warnings
164+
run: cargo clippy --workspace --exclude spirv-tools-ffi --exclude spirv-tools-cli --all-targets -- -D warnings

0 commit comments

Comments
 (0)