Skip to content

Commit 594d24f

Browse files
committed
Disable compilation of crates that use OptiX.
Because they don't work without OptiX, which is less important than CUDA and a pain to install. This means we don't have to manually exclude all these crates in CI, which is nice -- e.g. a vanilla `cargo build --workspace` works. Also, on Linux we don't need `--exclude "cudnn*"`. (We still need it on Windows.)
1 parent f5d6bd1 commit 594d24f

File tree

4 files changed

+93
-1947
lines changed

4 files changed

+93
-1947
lines changed

.github/workflows/ci_linux.yml

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -151,36 +151,21 @@ jobs:
151151
- name: Build workspace
152152
run: |
153153
docker exec "$CONTAINER_NAME" bash -lc 'set -euo pipefail
154-
cargo build --workspace \
155-
--exclude "optix*" \
156-
--exclude "path-tracer" \
157-
--exclude "denoiser" \
158-
--exclude "ex0*" \
159-
--exclude "cudnn*"
154+
cargo build
160155
'
161156
162157
- name: Clippy
163158
run: |
164159
docker exec "$CONTAINER_NAME" bash -lc 'set -euo pipefail
165160
export RUSTFLAGS=-Dwarnings
166-
cargo clippy --workspace \
167-
--exclude "optix*" \
168-
--exclude "path-tracer" \
169-
--exclude "denoiser" \
170-
--exclude "ex0*" \
171-
--exclude "cudnn*"
161+
cargo clippy
172162
'
173163
174164
- name: Check documentation
175165
run: |
176166
docker exec "$CONTAINER_NAME" bash -lc 'set -euo pipefail
177167
export RUSTDOCFLAGS=-Dwarnings
178168
cargo doc --workspace --all-features --document-private-items --no-deps \
179-
--exclude "optix*" \
180-
--exclude "path-tracer" \
181-
--exclude "denoiser" \
182-
--exclude "ex0*" \
183-
--exclude "cudnn*" \
184169
--exclude "cust_raw"
185170
'
186171

.github/workflows/ci_windows.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,12 @@ jobs:
8989
run: cargo build --all-features -p cust_raw
9090

9191
- name: Build workspace
92-
run: cargo build --workspace --exclude "optix*" --exclude "path-tracer" --exclude "denoiser" --exclude "vecadd*" --exclude "gemm*" --exclude "ex0*" --exclude "cudnn*" --exclude "sha2*"
92+
run: cargo build --workspace --exclude "cudnn*" --exclude "gemm*" --exclude "sha2*" --exclude "vecadd*"
9393

9494
- name: Clippy
9595
env:
9696
RUSTFLAGS: -Dwarnings
97-
run: cargo clippy --workspace --exclude "optix*" --exclude "path-tracer" --exclude "denoiser" --exclude "vecadd*" --exclude "gemm*" --exclude "ex0*" --exclude "cudnn*" --exclude "sha2*"
97+
run: cargo clippy --workspace --exclude "cudnn*" --exclude "gemm*" --exclude "sha2*" --exclude "vecadd*"
9898

9999
# Don't currently test because many tests rely on the system having a CUDA GPU
100100
# - name: Test
@@ -103,7 +103,7 @@ jobs:
103103
- name: Check documentation
104104
env:
105105
RUSTDOCFLAGS: -Dwarnings
106-
run: cargo doc --workspace --all-features --document-private-items --no-deps --exclude "optix*" --exclude "path-tracer" --exclude "denoiser" --exclude "vecadd*" --exclude "gemm*" --exclude "ex0*" --exclude "cudnn*" --exclude "sha2*" --exclude "cust_raw"
106+
run: cargo doc --workspace --all-features --document-private-items --no-deps --exclude "cudnn*" --exclude "cust_raw" --exclude "gemm*" --exclude "sha2*" --exclude "vecadd*"
107107

108108
# Disabled due to dll issues, someone with Windows knowledge needed
109109
# - name: Compiletest

0 commit comments

Comments
 (0)