Skip to content

Bump engine to api 37#186930

Closed
AbdeMohlbi wants to merge 6 commits into
flutter:masterfrom
AbdeMohlbi:bump_engine_to_api_37
Closed

Bump engine to api 37#186930
AbdeMohlbi wants to merge 6 commits into
flutter:masterfrom
AbdeMohlbi:bump_engine_to_api_37

Conversation

@AbdeMohlbi

@AbdeMohlbi AbdeMohlbi commented May 22, 2026

Copy link
Copy Markdown
Member

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 22, 2026
@github-actions github-actions Bot added platform-android Android applications specifically engine flutter/engine related. See also e: labels. team-android Owned by Android platform team labels May 22, 2026
@AbdeMohlbi AbdeMohlbi closed this May 22, 2026
@AbdeMohlbi AbdeMohlbi reopened this May 22, 2026
@fluttergithubbot

Copy link
Copy Markdown
Contributor

An existing Git SHA, 3bb4b000ae9ed812510c07204c9c7c975a3fbc3e, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

@github-actions github-actions Bot removed the CICD Run CI/CD label May 22, 2026
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 22, 2026
@github-actions github-actions Bot removed the CICD Run CI/CD label May 22, 2026
@AbdeMohlbi AbdeMohlbi added the CICD Run CI/CD label May 22, 2026
@reidbaker reidbaker requested a review from jesswrd May 25, 2026 15:58
@github-actions github-actions Bot removed the CICD Run CI/CD label May 26, 2026
@AbdeMohlbi AbdeMohlbi added the CICD Run CI/CD label May 26, 2026
Comment thread DEPS
# See tools/gradle/README.md for update instructions.
# Version here means the CIPD tag.
'version': 'version:8.11.1',
'version': 'version:9.4.1',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jesswrd I think you had thoughts on what this value should or should not be.

}
dependencies {
classpath "com.android.tools.build:gradle:8.11.1"
classpath "com.android.tools.build:gradle:9.2.1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this version not match the version you added in deps?

@AbdeMohlbi AbdeMohlbi May 26, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this version not match the version you added in deps?

Seems like a typo, i will fix it after i'm back

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @reidbaker, i checked again this is the used AGP version not the gradle version, if i did bump the compilesdk to 37, i get:

We recommend using a newer Android Gradle plugin to use compileSdk = 37

This Android Gradle plugin (8.9.1) was tested up to compileSdk = 36.

You are strongly encouraged to update your project to use a newer
Android Gradle plugin that has been tested with compileSdk = 37.

If you are already using the latest version of the Android Gradle plugin,
you may need to wait until a newer version with support for compileSdk = 37 is available.

which would require
classpath "com.android.tools.build:gradle:9.2.1"
and a gradle version of at least 9.4.1
which means these are correct values

@reidbaker

Copy link
Copy Markdown
Contributor

sdk 37 is uploaded to cipd (It was uploaded last week but I forgot to tag you)

@AbdeMohlbi

Copy link
Copy Markdown
Member Author

@reidbaker i think i also need gradle 9.4.1 because i see in logs:

________ running 'cipd ensure -log-level info -root /b/s/w/ir/cache/builder -ensure-file /b/s/w/ir/x/t/tmpbkq7ywit.ensure' in '.'
Errors:
  failed to resolve flutter/gradle@version:9.4.1 (line 41): no such tag
Error: Command 'cipd ensure -log-level info -root /b/s/w/ir/cache/builder -ensure-file /b/s/w/ir/x/t/tmpbkq7ywit.ensure' returned non-zero exit status 1

@github-actions github-actions Bot removed the CICD Run CI/CD label Jun 4, 2026
@AbdeMohlbi AbdeMohlbi added the CICD Run CI/CD label Jun 4, 2026
@reidbaker

Copy link
Copy Markdown
Contributor

@reidbaker i think i also need gradle 9.4.1 because i see in logs:

________ running 'cipd ensure -log-level info -root /b/s/w/ir/cache/builder -ensure-file /b/s/w/ir/x/t/tmpbkq7ywit.ensure' in '.'
Errors:
  failed to resolve flutter/gradle@version:9.4.1 (line 41): no such tag
Error: Command 'cipd ensure -log-level info -root /b/s/w/ir/cache/builder -ensure-file /b/s/w/ir/x/t/tmpbkq7ywit.ensure' returned non-zero exit status 1

Lets not update gradle that far as part of this work.
According to
https://developer.android.com/build/releases/agp-9-1-0-release-notes AGP 9.1 should work.
It looks like @jesswrd already added 9.1 last week https://chrome-infra-packages.appspot.com/p/flutter/gradle/+/iy63cJIShjCXlk7vAxmyzWPTPGKGfqDsIRKwaUqhYJsC

@AbdeMohlbi

Copy link
Copy Markdown
Member Author

I'm switching to another task, you can ignore the tasks related to this, @reidbaker thanks for the time and sorry i can't finish this one.

@AbdeMohlbi AbdeMohlbi closed this Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD engine flutter/engine related. See also e: labels. platform-android Android applications specifically team-android Owned by Android platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants