Skip to content

Commit 364cc2f

Browse files
Don't rely on SystemPrefix to set IOS_UNZIPPERED_TWIN_PREFIX_PATH
Mac Catalyst shouldn't have a SystemPrefix set in its SDK variant, so don't rely on that to set IOS_UNZIPPERED_TWIN_PREFIX_PATH, hard code it instead. Set MACOS_UNZIPPERED_TWIN_PREFIX_PATH too for completeness. rdar://165798693
1 parent dc3aecc commit 364cc2f

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

Sources/SWBCore/Settings/BuiltinMacros.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ public final class BuiltinMacros {
120120
public static let HOST_PLATFORM = BuiltinMacros.declareStringMacro("HOST_PLATFORM")
121121
public static let IOS_UNZIPPERED_TWIN_PREFIX_PATH = BuiltinMacros.declareStringMacro("IOS_UNZIPPERED_TWIN_PREFIX_PATH")
122122
public static let IPHONEOS_DEPLOYMENT_TARGET = BuiltinMacros.declareStringMacro("IPHONEOS_DEPLOYMENT_TARGET")
123+
public static let MACOS_UNZIPPERED_TWIN_PREFIX_PATH = BuiltinMacros.declareStringMacro("MACOS_UNZIPPERED_TWIN_PREFIX_PATH")
123124
public static let MACOSX_DEPLOYMENT_TARGET = BuiltinMacros.declareStringMacro("MACOSX_DEPLOYMENT_TARGET")
124125
public static let NATIVE_ARCH = BuiltinMacros.declareStringMacro("NATIVE_ARCH")
125126
public static let NATIVE_ARCH_32_BIT = BuiltinMacros.declareStringMacro("NATIVE_ARCH_32_BIT")
@@ -1993,6 +1994,7 @@ public final class BuiltinMacros {
19931994
MACOS_CREATOR_ARG,
19941995
MACOS_TYPE,
19951996
MACOS_TYPE_ARG,
1997+
MACOS_UNZIPPERED_TWIN_PREFIX_PATH,
19961998
MAC_OS_X_PRODUCT_BUILD_VERSION,
19971999
MAC_OS_X_VERSION_ACTUAL,
19982000
MAC_OS_X_VERSION_MAJOR,

Sources/SWBCore/Settings/Settings.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2524,6 +2524,11 @@ private class SettingsBuilder: ProjectMatchLookup {
25242524
platformTable.push(BuiltinMacros.PLATFORM_DEVELOPER_USR_DIR, Static { BuiltinMacros.namespace.parseString("$(DEVELOPER_USR_DIR)") })
25252525
platformTable.push(BuiltinMacros.PLATFORM_DEVELOPER_BIN_DIR, Static { BuiltinMacros.namespace.parseString("$(DEVELOPER_BIN_DIR)") })
25262526
platformTable.push(BuiltinMacros.PLATFORM_DEVELOPER_SDK_DIR, Static { BuiltinMacros.namespace.parseString("$(DEVELOPER_SDK_DIR)") })
2527+
2528+
// Set twin prefix paths in macOS for Mac Catalyst.
2529+
platformTable.push(BuiltinMacros.MACOS_UNZIPPERED_TWIN_PREFIX_PATH, literal: "")
2530+
platformTable.push(BuiltinMacros.IOS_UNZIPPERED_TWIN_PREFIX_PATH, literal: "/System/iOSSupport")
2531+
25272532
} else {
25282533
platformTable.push(BuiltinMacros.PLATFORM_DEVELOPER_APPLICATIONS_DIR, literal: "\(platform.path.str)/Developer/Applications")
25292534
platformTable.push(BuiltinMacros.PLATFORM_DEVELOPER_TOOLS_DIR, literal: "\(platform.path.str)/Developer/Tools")
@@ -2585,11 +2590,6 @@ private class SettingsBuilder: ProjectMatchLookup {
25852590
sdkTable.pushContentsOf(defaultSettingsTable)
25862591
}
25872592

2588-
// Set IOS_UNZIPPERED_TWIN_PREFIX_PATH to the Mac Catalyst variant's prefix path, even for the macOS variant.
2589-
if let macCatalystVariant = sdk.variant(for: MacCatalystInfo.sdkVariantName) {
2590-
sdkTable.push(BuiltinMacros.IOS_UNZIPPERED_TWIN_PREFIX_PATH, literal: macCatalystVariant.systemPrefix)
2591-
}
2592-
25932593
// Add the settings provided by the SDK variant, if there is one.
25942594
if let variant {
25952595
// Late-bound by `SDKRegistry.loadExtendedInfo` and may be nil if an error (which will have already been reported) was encountered during loading.

0 commit comments

Comments
 (0)