-
Notifications
You must be signed in to change notification settings - Fork 28.5k
Exclude arm64 from valid iOS simulators #73828
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 t 8000 o your account
Conversation
25831f6
to
88b903e
Compare
'objc', | ||
'hello', | ||
], workingDirectory: tempDir.path); | ||
group('iOS app validation', () { |
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.
Most of this file is whitespace changes.
timeout: const Timeout(Duration(minutes: 5)), | ||
); | ||
} | ||
testWithoutContext('build for simulator with all available architectures', () { |
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 test is new.
Third time's the charm (maybe)! |
Can you remind me what the failure was last time? Xcode version issues? |
Xcode 11 had an allow list s.pod_target_xcconfig = { 'DEFINES_MODULE' => 'YES', 'VALID_ARCHS[sdk=iphonesimulator*]' => 'x86_64' } Xcode 12 has a new When both were present, Xcode 11 had a little freak out, which was the cause for the last two reverts (this took me hours to figure out).
That error doesn't even make sense--the But now the devicelab is on Xcode 12 and it's the minimum version the tool allows for developers. Xcode 12 handles these build settings, so with this PR even when plugins have |
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!
Another attempt at #73458 on top of minimum Xcode 12 version bump.
Description
Exclude unsupported new arm64 and old i386 iOS simulators at the build setting layer. This will allow the excluded architectures to be excluded when the project is run from Xcode, instead of just from the
flutter
command line.Do this for plugin targets (in the podhelper) as well as in the Runner app target (Generated.xcconfig).
Update the add-to-app host apps to exclude arm64 simulators since Flutter still doesn't contain that slice.
Sample code update: flutter/samples#646
Website docs update: flutter/website#5174
#64502 (comment)
Added a ios_content_validation_test case to target the simulator with
ONLY_ACTIVE_ARCH=NO
which will target all valid architectures that have not been explicitly excluded. This test case would fail if arm64 or i386 were not excluded.