fix: BuildPlayer exception handling

siren403 · Jun 28, 2023 · 72b1449 · 72b1449
1 parent 8e076e1
commit 72b1449
Showing 1 changed file with 16 additions and 6 deletions.
diff --git a/Packages/UActions/Editor/Actions/Build.cs b/Packages/UActions/Editor/Actions/Build.cs
@@ -2,6 +2,7 @@
 using System.IO;
 using System.Linq;
 using UnityEditor;
+using UnityEditor.Build.Reporting;
 using UnityEngine;
 using YamlDotNet.Core;
 using YamlDotNet.Serialization;
@@ -74,13 +75,22 @@ public void Execute(IWorkflowContext context)
                 options = additional,
             };
 
-            if (!context.With.Is(InputSkipBuild))
+            if (context.With.Is(InputSkipBuild)) return;
+
+            BuildReport report = null;
+            try
+            {
+                report = BuildPipeline.BuildPlayer(options);
+            }
+            catch (Exception e)
+            {
+                context.Log(e.Message);
+                EditorApplication.Exit(1);
+            }
+
+            if (!Application.isBatchMode && report != null)
             {
-                var report = BuildPipeline.BuildPlayer(options);
-                if (!Application.isBatchMode)
-                {
-                    ActionHelper.OpenFolder(report.summary.outputPath);
-                }
+                ActionHelper.OpenFolder(report.summary.outputPath);
             }
         }