Don't fail when QNX 7 SDP is present on the system and Rosetta is not installed

jakepetroules · jakepetroules · commit cfe366a7c48f · 2025-10-02T16:45:37.000-07:00
Auxiliary platform lookup is intended to be failable; one of the three calls to cachedQNXSDPInstallations was missing a `try?` Closes #836
diff --git a/Sources/SWBQNXPlatform/Plugin.swift b/Sources/SWBQNXPlatform/Plugin.swift
@@ -45,7 +45,7 @@ struct QNXEnvironmentExtension: EnvironmentExtension {
     let plugin: QNXPlugin
 
     func additionalEnvironmentVariables(context: any EnvironmentExtensionAdditionalEnvironmentVariablesContext) async throws -> [String : String] {
-        if let latest = try await plugin.cachedQNXSDPInstallations(host: context.hostOperatingSystem).first {
+        if let latest = try? await plugin.cachedQNXSDPInstallations(host: context.hostOperatingSystem).first {
             return .init(latest.environment)
         }
         return [:]

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ struct QNXEnvironmentExtension: EnvironmentExtension {`
`45`	`45`	`let plugin: QNXPlugin`
`46`	`46`
`47`	`47`	`func additionalEnvironmentVariables(context: any EnvironmentExtensionAdditionalEnvironmentVariablesContext) async throws -> [String : String] {`
`48`		`- if let latest = try await plugin.cachedQNXSDPInstallations(host: context.hostOperatingSystem).first {`
	`48`	`+ if let latest = try? await plugin.cachedQNXSDPInstallations(host: context.hostOperatingSystem).first {`
`49`	`49`	`return .init(latest.environment)`
`50`	`50`	`}`
`51`	`51`	`return [:]`