clippy: add cargo gpu clippy

Author: Firestar99

crates/cargo-gpu/src/build.rs

@@ -21,14 +21,19 @@ pub struct BuildArgs {
     #[clap(long, short, action)]
     pub watch: bool,
 
-    /// the flattened [`SpirvBuilder`]
+    /// The flattened [`SpirvBuilder`]
     #[clap(flatten)]
     #[serde(flatten)]
     pub spirv_builder: SpirvBuilder,
 
-    ///Renames the manifest.json file to the given name
+    /// Renames the `manifest.json` file to the given name
     #[clap(long, short, default_value = "manifest.json")]
     pub manifest_file: String,
+
+    /// When building fails with [`SpirvBuilderError::NoArtifactProduced`], count it as a success anyway.
+    /// Used for e.g. `clippy`, which doesn't produce any artifacts. Defaults to false.
+    #[clap(skip)]
+    pub allow_no_artifacts: bool,
 }
 
 impl Default for BuildArgs {
@@ -39,6 +44,7 @@ impl Default for BuildArgs {
             watch: false,
             spirv_builder: SpirvBuilder::default(),
             manifest_file: String::from("manifest.json"),
+            allow_no_artifacts: false,
         }
     }
 }
@@ -116,8 +122,16 @@ impl Build {
                 "Compiling shaders at {}...\n",
                 self.install.shader_crate.display()
             );
-            let result = self.build.spirv_builder.build()?;
-            self.parse_compilation_result(&result)?;
+            let result = self.build.spirv_builder.build();
+            match result {
+                Ok(result) => {
+                    self.parse_compilation_result(&result)?;
+                }
+                // conditionally ignore NoArtifactProduced
+                Err(SpirvBuilderError::NoArtifactProduced { .. })
+                    if self.build.allow_no_artifacts => {}
+                Err(err) => return Err(err.into()),
+            }
         }
         Ok(())
     }

crates/cargo-gpu/src/config.rs

@@ -31,7 +31,7 @@ impl Config {
     ) -> anyhow::Result<crate::build::Build> {
         let mut config = metadata.as_json(shader_crate_path)?;
 
-        env_args.retain(|arg| !(arg == "build" || arg == "install"));
+        env_args.retain(|arg| !(arg == "build" || arg == "install" || arg == "clippy"));
         let cli_args_json = Self::cli_args_to_json(env_args)?;
         Self::json_merge(&mut config, cli_args_json, None)?;
 

crates/cargo-gpu/src/lib.rs

@@ -105,6 +105,9 @@ pub enum Command {
     /// Compile a shader crate to SPIR-V.
     Build(Box<Build>),
 
+    /// Run clippy on a shader crate with a SPIR-V target
+    Clippy(Box<Build>),
+
     /// Show some useful values.
     Show(Show),
 
@@ -140,13 +143,17 @@ impl Command {
                 );
                 command.install.run()?;
             }
-            Self::Build(build) => {
+            Self::Build(build) | Self::Clippy(build) => {
                 let shader_crate_path = &build.install.shader_crate;
                 let mut command = config::Config::clap_command_with_cargo_config(
                     shader_crate_path,
                     env_args,
                     metadata_cache,
                 )?;
+                if let Self::Clippy(_) = self {
+                    command.build.spirv_builder.cargo_cmd = Some("clippy".into());
+                    command.build.allow_no_artifacts = true;
+                }
                 log::debug!("building with final merged arguments: {command:#?}");
 
                 if command.build.watch {