-
Notifications
You must be signed in to change notification settings - Fork 28.7k
[ Tool ] Remove support for Android x86 targets #169884
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
Dart x86 is deprecated, no longer shipped, and will eventually be removed from the Dart VM. This PR removes the last remaining x86 target, android-x86, from the tool. Work towards #157543
|
@@ -27,15 +27,13 @@ Future<void> main() async { | ||
...baseApkFiles, | ||
'lib/armeabi-v7a/libflutter.so', | ||
'lib/arm64-v8a/libflutter.so', | ||
// Debug mode intentionally includes `x86` and `x86_64`. | ||
'lib/x86/libflutter.so', | ||
// Debug mode intentionally includes `x86_64`. |
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.
Not sure what this comment is supposed to mean ?
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 was mostly a mechanical update, so I'm not sure either... 😅 it can probably be removed.
Do you happen to know where |
Let me take a look |
Looks like it is here: Lines 628 to 657 in a79827a
And runs a custom recipe. |
I did come across that, but it's not clear what's actually being run. |
Oh, I found it. I'll send you a recipes CL. |
Dart x86 is deprecated, no longer shipped, and will eventually be removed from the Dart VM. This PR removes the last remaining x86 target, android-x86, from the tool.
Work towards #157543 and #160932