fix(URP): fix missing shader on URP build (#30)

* fix(URP): fix missing shader on URP build * docs: bump Unity package version * fix(URP): don't blit on null material
happy-turtle · Apr 26, 2024 · 91cdcf8 · 91cdcf8
1 parent 863657d
commit 91cdcf8
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 5 deletions.
diff --git a/Shaders/Resources/OitRenderURP.shader b/Shaders/Resources/OitRenderURP.shader
@@ -5,7 +5,7 @@ Shader "Hidden/OitRenderURP"
 		PackageRequirements {
 			"com.unity.render-pipelines.universal"
 		}
-        Tags { "RenderPipeline" = "UniversalRenderPipeline" }
+        Tags { "RenderPipeline" = "UniversalPipeline" }
 		Pass {
             Name "URP Order-Independent Transparency Pass"
 			ZTest Always

diff --git a/Shared/OitLinkedList.cs b/Shared/OitLinkedList.cs
@@ -47,6 +47,10 @@ public void PreRender(CommandBuffer command)
         public Material Render(CommandBuffer command, RenderTargetIdentifier src, RenderTargetIdentifier dest)
         {
             command.ClearRandomWriteTargets();
+            if (linkedListMaterial == null)
+            {
+                return null;
+            }
             // blend linked list
             linkedListMaterial.SetBuffer(fragmentLinkBufferId, fragmentLinkBuffer);
             linkedListMaterial.SetBuffer(startOffsetBufferId, startOffsetBuffer);

diff --git a/URP/Runtime/OitPass.cs b/URP/Runtime/OitPass.cs
@@ -30,8 +30,11 @@ public override void Execute(ScriptableRenderContext context, ref RenderingData
             CommandBuffer cmd = CommandBufferPool.Get("Order Independent Transparency");
             var mat = orderIndependentTransparency.Render(cmd, renderingData.cameraData.renderer.cameraColorTargetHandle,
                 renderingData.cameraData.renderer.cameraColorTargetHandle);
-            Blitter.BlitCameraTexture(cmd, renderingData.cameraData.renderer.cameraColorTargetHandle,
-                renderingData.cameraData.renderer.cameraColorTargetHandle, mat, 0);
+            if (mat != null)
+            {
+                Blitter.BlitCameraTexture(cmd, renderingData.cameraData.renderer.cameraColorTargetHandle,
+                    renderingData.cameraData.renderer.cameraColorTargetHandle, mat, 0);
+            }
             context.ExecuteCommandBuffer(cmd);
             cmd.Clear();
             CommandBufferPool.Release(cmd);

diff --git a/package.json b/package.json
@@ -1,14 +1,14 @@
 {
   "name": "org.happy-turtle.order-independent-transparency",
   "version": "5.0.0",
-  "unity": "2021.2",
+  "unity": "2022.3",
   "displayName": "Order-independent Transparency",
   "description": "This is an implementation of order-independent transparency for the Built-In Pipeline. It uses Per-Pixel Linked Lists, implemented with RWStructuredBuffers. This is a feature requiring Shader Model 5.0 with ComputeBuffers, see the Unity Manual for supported platforms.",
   "author": {
     "name": "Till Davin",
     "email": "code@tilldavin.de"
   },
-  "dependencies": {},
+  "dependencies": { },
   "samples": [
     {
       "displayName": "Post Process Demo",