-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[Xcode] Use the Legacy Build System to use Swift 5 mode on Xcode 10 #1826
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This works around the Swift version validation by the New Build System. See the following for the details: - https://learningswift.brightdigit.com/swift-5-0-xcode-10-1/ - https://forums.swift.org/t/how-to-set-swift-version-5-for-recent-dev-snapshots-in-xcode-build-settings/18692
@swift-ci Please test |
@@ -2739,6 +2739,7 @@ | |||
ONLY_ACTIVE_ARCH = YES; | |||
SDKROOT = macosx; | |||
SWIFT_ACTIVE_COMPILATION_CONDITIONS = "NS_FOUNDATION_ALLOWS_TESTABLE_IMPORT DEBUG"; | |||
SWIFT_VERSION = 5.0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now set at the project level, not at per-target level.
Hm, do we want to switch to 5.0 mode yet? |
IIUC it's already built with 5.0 mode on Linux (CMake build). |
Hm the Xcode project of swift-corelibs-xctest should also be updated or we should have a condition: #if swift(>=5.0)
guard let parsingTests = testRoot[kURLTestParsingTestsKey] as? [Any] else {
XCTFail("Unable to create the parsingTests dictionary")
return nil
}
#else
guard let parsingTests = testRoot?[kURLTestParsingTestsKey] as? [Any] else {
XCTFail("Unable to create the parsingTests dictionary")
return nil
}
#endif |
Closing this in favor of #1829. |
This works around the Swift version validation by the New Build System. See the following for the details:
Supersedes #1818.