finally

bitsandfoxes · bitsandfoxes · commit 0181a41abe00 · 2025-02-19T10:02:53.000+01:00
diff --git a/src/Sentry.Unity.Editor/Native/BuildPostProcess.cs b/src/Sentry.Unity.Editor/Native/BuildPostProcess.cs
@@ -69,8 +69,11 @@ public static void OnPostProcessBuild(BuildTarget target, string executablePath)
             logger.LogError(e, "Failed to add the Sentry native integration to the built application");
             throw new BuildFailedException("Sentry Native BuildPostProcess failed");
         }
-
-        UploadDebugSymbols(logger, target, buildOutputDir, executableName, options, cliOptions, isMono);
+        finally
+        {
+            // We always want to end up with the debug symbols being uploadde
+            UploadDebugSymbols(logger, target, buildOutputDir, executableName, options, cliOptions, isMono);
+        }
     }
 
     private static bool IsEnabledForPlatform(BuildTarget target, SentryUnityOptions options) => target switch

Original file line number	Diff line number	Diff line change
`@@ -69,8 +69,11 @@ public static void OnPostProcessBuild(BuildTarget target, string executablePath)`
`69`	`69`	`logger.LogError(e, "Failed to add the Sentry native integration to the built application");`
`70`	`70`	`throw new BuildFailedException("Sentry Native BuildPostProcess failed");`
`71`	`71`	`}`
`72`		`-`
`73`		`- UploadDebugSymbols(logger, target, buildOutputDir, executableName, options, cliOptions, isMono);`
	`72`	`+ finally`
	`73`	`+ {`
	`74`	`+ // We always want to end up with the debug symbols being uploadde`
	`75`	`+ UploadDebugSymbols(logger, target, buildOutputDir, executableName, options, cliOptions, isMono);`
	`76`	`+ }`
`74`	`77`	`}`
`75`	`78`
`76`	`79`	`private static bool IsEnabledForPlatform(BuildTarget target, SentryUnityOptions options) => target switch`