Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@GaryQian
Copy link
Contributor

@GaryQian GaryQian commented Aug 13, 2020

Fixes flutter/flutter#56597

This upgrades to default to building with Android 30. Also uses platform-tools 30.0.4 and build-tools 30.0.1

embedding_deps_dir = os.path.join(buildroot_dir, 'third_party', 'android_embedding_dependencies', 'lib')
classpath = map(str, [
os.path.join(buildroot_dir, 'third_party', 'android_tools', 'sdk', 'platforms', 'android-29', 'android.jar'),
os.path.join(buildroot_dir, 'third_party', 'android_tools', 'sdk', 'platforms', 'android-30', 'android.jar'),
Copy link
Member

Choose a reason for hiding this comment

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

@xster
Copy link
Member

xster commented Aug 13, 2020

LGTM, thanks

@GaryQian
Copy link
Contributor Author

GaryQian commented Aug 14, 2020

Anyone know why https://ci.chromium.org/p/flutter/builders/try/Windows%20Host%20Engine/11910? vertices_unittests.cc VerticesAccuratelyReportsSize seems to keep timing out? This does not repro locally. Windows host debug unopt should be unrelated to this change.

@GaryQian
Copy link
Contributor Author

flutter/buildroot@97f6f00 is probably causing this. cc @stuartmorgan

@stuartmorgan-g
Copy link
Contributor

Sorry, everything was green in my test run before landing that buildroot change, but then once I actually tried to roll with the real builroot it's been failing unit tests (I suspect it's throwing an exception, which the Windows CI seems to treat as a hang instead). Feel free to revert the VS 2019 change to unblock your roll.

@GaryQian GaryQian added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Aug 16, 2020
@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Mac iOS Engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@fluttergithubbot fluttergithubbot removed the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Aug 16, 2020
@GaryQian
Copy link
Contributor Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate engine to target Android SDK 30

6 participants