Merge pull request #1346 from ahoppen/reveal-diag-bundle

ahoppen · web-flow · commit 6103661636a8 · 2024-05-24T12:37:20.000-07:00
On macOS, reveal diagnostic bundle in Finder after it has been created
diff --git a/Sources/Diagnose/DiagnoseCommand.swift b/Sources/Diagnose/DiagnoseCommand.swift
@@ -352,6 +352,15 @@ public struct DiagnoseCommand: AsyncParsableCommand {
       """
     )
 
+    #if os(macOS)
+    // Reveal the bundle in Finder on macOS
+    do {
+      let process = try Process.launch(arguments: ["open", "-R", bundlePath.path], workingDirectory: nil)
+      try await process.waitUntilExitSendingSigIntOnTaskCancellation()
+    } catch {
+      // If revealing the bundle in Finder should fail, we don't care. We still printed the bundle path to stdout.
+    }
+    #endif
   }
 
   @MainActor

Original file line number	Diff line number	Diff line change
`@@ -352,6 +352,15 @@ public struct DiagnoseCommand: AsyncParsableCommand {`
`352`	`352`	`"""`
`353`	`353`	`)`
`354`	`354`
	`355`	`+ #if os(macOS)`
	`356`	`+ // Reveal the bundle in Finder on macOS`
	`357`	`+ do {`
	`358`	`+ let process = try Process.launch(arguments: ["open", "-R", bundlePath.path], workingDirectory: nil)`
	`359`	`+ try await process.waitUntilExitSendingSigIntOnTaskCancellation()`
	`360`	`+ } catch {`
	`361`	`+ // If revealing the bundle in Finder should fail, we don't care. We still printed the bundle path to stdout.`
	`362`	`+ }`
	`363`	`+ #endif`
`355`	`364`	`}`
`356`	`365`
`357`	`366`	`@MainActor`