[CI] Use Xcode 14.2 #1820
Merged
[CI] Use Xcode 14.2 #1820
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BalestraPatrick
requested review from
brentleyjones,
keith,
segiddins and
thii
as code owners
BalestraPatrick
force-pushed
the
override-xcode-version
branch
3 times, most recently
from
183770e to
ee010ee
Compare
|
In the end, we don't want this flag, and just want our tests to work on either Xcode 13 or Xcode 14, right? Especially since we might be routed to either set of macs (or will we always be routed to Xcode 14 if we don't specify arm64?). |
|
If we want to keep running on x86_64, we need to run on iMacs for the moment. From this comment, those machines have Xcode 13.0, 13.4.1 and 14.2 installed, but given they run Ventura now, only 14.2 is usable. |
BalestraPatrick
force-pushed
the
override-xcode-version
branch
from
ee010ee to
2ecc20e
Compare
brentleyjones
approved these changes
Feb 8, 2023
This was referenced Feb 8, 2023
BalestraPatrick
added a commit
to bazelbuild/rules_swift
that referenced
this pull request
Feb 8, 2023
BalestraPatrick
added a commit
to bazelbuild/apple_support
that referenced
this pull request
Feb 9, 2023
tymurmustafaiev
pushed a commit
to tymurmustafaiev/rules_swift
that referenced
this pull request
Jul 19, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Looks like CI
iswas broken because the CI fleet was upgraded to Xcode 14.2 and some machines were missing the watchOS and tvOS SDKs. Now that the iMacs are properly set up, we should be able to simply use Xcode 14.2.We can't use Xcode 13.0 anymore because it doesn't run on macOS Ventura (which all the x86_64 machines are on after the upgrade).
Future tasks might include adding support for arm64 in our tooling so that we can switch to run on Mac Studios instead of iMacs.