Skip to content

Commit 4b8519a

Browse files
committed
Plumb the environment through to SettingsBuilderExtension.addSDKOverridingSettings().
rdar://159601154
1 parent 7150a57 commit 4b8519a

File tree

3 files changed

+4
-3
lines changed

3 files changed

+4
-3
lines changed

Sources/SWBApplePlatform/Plugin.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ struct AppleSettingsBuilderExtension: SettingsBuilderExtension {
264264
}
265265
func addOverrides(fromEnvironment: [String : String], parameters: BuildParameters) throws -> [String : String] { [:] }
266266
func addProductTypeDefaults(productType: ProductTypeSpec) -> [String : String] { [:] }
267-
func addSDKOverridingSettings(_ sdk: SDK, _ variant: SDKVariant?, _ sparseSDKs: [SDK], specLookupContext: any SWBCore.SpecLookupContext) throws -> [String : String] { [:] }
267+
func addSDKOverridingSettings(_ sdk: SDK, _ variant: SDKVariant?, _ sparseSDKs: [SDK], specLookupContext: any SWBCore.SpecLookupContext, environment: [String: String]) throws -> [String : String] { [:] }
268268
func addPlatformSDKSettings(_ platform: SWBCore.Platform?, _ sdk: SDK, _ sdkVariant: SDKVariant?) -> [String : String] { [:] }
269269
func xcconfigOverrideData(fromParameters: BuildParameters) -> ByteString { ByteString() }
270270
func getTargetTestingSwiftPluginFlags(_ scope: MacroEvaluationScope, toolchainRegistry: ToolchainRegistry, sdkRegistry: SDKRegistry, activeRunDestination: RunDestinationInfo?, project: SWBCore.Project?) -> [String] { [] }

Sources/SWBCore/Extensions/SettingsBuilderExtension.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public protocol SettingsBuilderExtension {
3737
func addSDKSettings(_ sdk: SDK, _ variant: SDKVariant?, _ sparseSDKs: [SDK]) throws -> [String : String]
3838

3939
/// Provides a table of overriding SDK settings
40-
func addSDKOverridingSettings(_ sdk: SDK, _ variant: SDKVariant?, _ sparseSDKs: [SDK], specLookupContext: any SpecLookupContext) throws -> [String: String]
40+
func addSDKOverridingSettings(_ sdk: SDK, _ variant: SDKVariant?, _ sparseSDKs: [SDK], specLookupContext: any SpecLookupContext, environment: [String: String]) throws -> [String: String]
4141

4242
/// Provides a table of default platform SDK settings
4343
func addPlatformSDKSettings(_ platform: Platform?, _ sdk: SDK, _ sdkVariant: SDKVariant?) -> [String: String]

Sources/SWBCore/Settings/Settings.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1497,9 +1497,10 @@ private class SettingsBuilder {
14971497

14981498
addSDKOverridingSettings(sdk, boundProperties.sdkVariant)
14991499

1500+
let environment = workspaceContext.userInfo?.buildSystemEnvironment ?? [:]
15001501
for settingsExtension in core.pluginManager.extensions(of: SettingsBuilderExtensionPoint.self) {
15011502
do {
1502-
let overridingSettings = try settingsExtension.addSDKOverridingSettings(sdk, boundProperties.sdkVariant, boundProperties.sparseSDKs, specLookupContext: specLookupContext)
1503+
let overridingSettings = try settingsExtension.addSDKOverridingSettings(sdk, boundProperties.sdkVariant, boundProperties.sparseSDKs, specLookupContext: specLookupContext, environment: environment)
15031504
pushTable(.exported) {
15041505
$0.pushContentsOf(createTableFromUserSettings(overridingSettings))
15051506
}

0 commit comments

Comments
 (0)