Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[ 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

Merged
merged 4 commits into from
Jun 4, 2025
Merged

Conversation

bkonyi
Copy link
Contributor

@bkonyi bkonyi commented Jun 2, 2025

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

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
@bkonyi bkonyi requested a review from a-siva June 2, 2025 20:19
@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Jun 2, 2025
@bkonyi bkonyi requested a review from matanlurey June 2, 2025 22:42
@bkonyi bkonyi marked this pull request as ready for review June 2, 2025 22:42
@matanlurey
Copy link
Contributor

 "android-x86" is not an allowed value for option "--target-platform".
 Run 'flutter -h' (or 'flutter <command> -h') for available flutter commands and options.

@@ -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`.
Copy link
Contributor

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 ?

Copy link
Contributor Author

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.

@bkonyi
Copy link
Contributor Author

bkonyi commented Jun 2, 2025

 "android-x86" is not an allowed value for option "--target-platform".
 Run 'flutter -h' (or 'flutter <command> -h') for available flutter commands and options.

Do you happen to know where Linux firebase_release_smoke_test lives @matanlurey? I couldn't find anything in flutter/flutter

@matanlurey
Copy link
Contributor

Let me take a look

@matanlurey
Copy link
Contributor

Looks like it is here:

flutter/.ci.yaml

Lines 628 to 657 in a79827a

- name: Linux firebase_release_smoke_test
recipe: firebaselab/firebaselab
timeout: 60
properties:
dependencies: >-
[
{"dependency": "android_sdk", "version": "version:36v1"},
{"dependency": "open_jdk", "version": "version:21"}
]
tags: >
["firebaselab"]
task_name: release_smoke_test
physical_devices: >-
[
"--device", "model=shiba,version=34",
"--device", "model=redfin,version=30",
"--device", "model=griffin,version=24"
]
# TODO(flutter/flutter#123331): This device is flaking.
# "--device", "model=Nexus6P,version=25"
virtual_devices: >-
[
"--device", "model=Nexus5.gce_x86,version=21",
"--device", "model=Nexus5.gce_x86,version=22",
"--device", "model=Nexus5.gce_x86,version=23",
"--device", "model=Nexus6P,version=25",
"--device", "model=Nexus6P,version=26",
"--device", "model=Nexus6P,version=27",
"--device", "model=NexusLowRes,version=29"
]

And runs a custom recipe.

@bkonyi
Copy link
Contributor Author

bkonyi commented Jun 2, 2025

Looks like it is here:

flutter/.ci.yaml

Lines 628 to 657 in a79827a

- name: Linux firebase_release_smoke_test
recipe: firebaselab/firebaselab
timeout: 60
properties:
dependencies: >-
[
{"dependency": "android_sdk", "version": "version:36v1"},
{"dependency": "open_jdk", "version": "version:21"}
]
tags: >
["firebaselab"]
task_name: release_smoke_test
physical_devices: >-
[
"--device", "model=shiba,version=34",
"--device", "model=redfin,version=30",
"--device", "model=griffin,version=24"
]
# TODO(flutter/flutter#123331): This device is flaking.
# "--device", "model=Nexus6P,version=25"
virtual_devices: >-
[
"--device", "model=Nexus5.gce_x86,version=21",
"--device", "model=Nexus5.gce_x86,version=22",
"--device", "model=Nexus5.gce_x86,version=23",
"--device", "model=Nexus6P,version=25",
"--device", "model=Nexus6P,version=26",
"--device", "model=Nexus6P,version=27",
"--device", "model=NexusLowRes,version=29"
]

And runs a custom recipe.

I did come across that, but it's not clear what's actually being run. --target-platform=android-x86 is coming from somewhere... 😞

@matanlurey
Copy link
Contributor

Oh, I found it. I'll send you a recipes CL.

@matanlurey
Copy link
Contributor

https://flutter-review.googlesource.com/c/recipes/+/66160

@bkonyi
Copy link
Contributor Author

bkonyi commented Jun 3, 2025

@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 4, 2025
@auto-submit auto-submit bot added this pull request to the merge queue Jun 4, 2025
Merged via the queue into master with commit c46d410 Jun 4, 2025
156 checks passed
@auto-submit auto-submit bot deleted the remove_x86_support branch June 4, 2025 01:27
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 4, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool Affects the "flutter" command-line tool. See also t: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.