Fix for failing Windows build

For some reason `spirv-tools-sys` only fails to build on windows when
run from inside `cargo test`. Maybe it's something to do with what gets
set in ENV?

Also added `just` so that we have a canonical way of running CI-related
tasks.

Author: tombh

.github/workflows/push.yaml

@@ -28,15 +28,20 @@ jobs:
     steps:
       - uses: actions/checkout@v2
       - uses: moonrepo/setup-rust@v1
+      - uses: extractions/setup-just@v2
       - name: Install Rust toolchain
         run: |
           rustup default stable
           rustup update
       - run: cargo test
+      - name: Run a full build
+        run: just build-shader-template
+          
 
-  clippy:
+  lints:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - uses: moonrepo/setup-rust@v1
-      - run: cargo clippy -- --deny warnings
+      - uses: extractions/setup-just@v2
+      - run: just lints

crates/cargo-gpu/src/build.rs

@@ -161,13 +161,17 @@ mod test {
             &format!("{}", output_dir.display()),
         ];
         if let Cli {
-            command: Command::Build(mut build),
+            command: Command::Build(build),
         } = Cli::parse_from(args)
         {
             assert_eq!(shader_crate_path, build.install.shader_crate);
             assert_eq!(output_dir, build.output_dir);
 
-            build.run();
+            // TODO:
+            // For some reason running a full build (`build.run()`) inside tests fails on Windows.
+            // The error is in the `build.rs` step of compiling `spirv-tools-sys`. It is not clear
+            // from the logged error what the problem is. For now we'll just run a full build
+            // outside the tests environment.
         } else {
             panic!("was not a build command");
         }

crates/cargo-gpu/src/main.rs

@@ -83,11 +83,11 @@ fn main() {
         }
         Command::Build(mut build) => {
             log::debug!("building with arguments: {build:#?}");
-            build.run()
+            build.run();
         }
         Command::Toml(toml) => {
             log::debug!("building by toml file with arguments: {toml:#?}");
-            toml.run()
+            toml.run();
         }
         Command::Show(show) => show.run(),
         Command::DumpUsage => dump_full_usage_for_readme(),

justfile

@@ -0,0 +1,13 @@
+[group: 'ci']
+build-shader-template:
+  cargo install --path crates/cargo-gpu
+  cargo gpu install --shader-crate crates/shader-crate-template
+  cargo gpu build --shader-crate crates/shader-crate-template --output-dir test-shaders
+  ls -lah test-shaders
+  cat test-shaders/manifest.json
+
+[group: 'ci']
+lints:
+  cargo clippy -- --deny warnings
+  cargo fmt --check
+