Skip to content

Commit

Permalink
fix: renderling crate doesn't need to by dylib (#123)
Browse files Browse the repository at this point in the history
* fix: renderling crate doesn't need to by dylib

* github runners: separate shader validation steps

* try rlib
  • Loading branch information
schell authored Aug 22, 2024
1 parent 6825a06 commit 3a70bd5
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/push.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,10 @@ jobs:
- uses: Swatinem/rust-cache@v2
with:
workspaces: shaders
- run: |
# ensure the shader binaries were properly checked in
rm -rf crates/renderling/src/linkage/*.spv
cd shaders && cargo run --release && cd ..
git diff --exit-code --no-ext-diff crates/renderling/src/linkage
# ensure the shader binaries were properly checked in
- run: rm -rf crates/renderling/src/linkage/*.spv
- run: cd shaders && cargo run --release && cd ..
- run: git diff --exit-code --no-ext-diff crates/renderling/src/linkage

renderling-clippy:
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion crates/renderling/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ readme = "../../README.md"
# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[lib]
crate-type = ["lib", "dylib"]
crate-type = ["rlib", "cdylib"]

[features]
default = ["gltf", "sdf", "shaders", "tutorial", "winit"]
Expand Down

0 comments on commit 3a70bd5

Please sign in to comment.