-
Notifications
You must be signed in to change notification settings - Fork 18
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
Added support for XCode 14.2, iOS 16.2 and macOS 13.1 #264
Conversation
env.put("SYSROOT", "/opt/platformsdk/MacOSX11.3.sdk"); | ||
env.put("MACOS_13_VERSION", "13.1"); | ||
env.put("XCODE_14_CLANG_VERSION", "14.0.0"); | ||
env.put("SWIFT_5_5_VERSION", "5.5"); |
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.
is it swift 5.5 not 5.7? https://developer.apple.com/documentation/xcode-release-notes/xcode-14_2-release-notes
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.
It's a folder name. There is no "swift-5.7" in Xcode. Only "swift", "swift-5.0" and "swift-5.5"
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.
LGTM!
@@ -472,21 +472,18 @@ COPY template.local.properties ${EXTENSION_LOCAL_PROPERTIES_TEMPLATE} | |||
# we still need to be able to resolve the variables in the case when we run the docker container locally when debugging | |||
|
|||
# NOTE: These values are replicated in server/scripts/standalone/service-standalone.sh | |||
# Install flow from >=1.2.174 |
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.
👍
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.
Looks good!
@@ -39,11 +33,22 @@ export XCODE_13_CLANG_VERSION=13.0.0 | |||
export SWIFT_5_5_VERSION=5.5 | |||
export IOS_VERSION_MIN=9.0 | |||
export MACOS_VERSION_MIN=10.7 | |||
export LIB_TAPI_1_6_PATH=/usr/local/tapi1.6/lib |
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.
Do we really use it somewhere? I thought it's something we use like alternative to the official (xcode) tool chain on linux?
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.
No, we don't use it per se, but there are some old sdks (build.yml) that still reference it.
No description provided.