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

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Mar 8, 2019

The plan is to start using CIPD packages for this stuff, which gclient/depot_tools more natively supports (and will allow us to keep using versions of the NDK even if it's pulled from the Android repo).

I'd like to break that up into an NDK piece (Which is more critical) and an SDK piece.

Having the tool in the engine repo will allow me to iterate on that more easily. I'll open a separate PR to remove it from buildroot.

/cc @cbracken @jason-simmons @chinmaygarde

@dnfield dnfield merged commit 4c94049 into flutter:master Mar 8, 2019
@dnfield dnfield deleted the move_android_sdk_downloader branch March 8, 2019 20:10
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 8, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Mar 9, 2019
flutter/engine@6031407...87edd94

git log 6031407..87edd94 --no-merges --oneline
87edd94 Add read-only persistent cache (flutter/engine#8049)
4c94049 Move android_sdk_downloader so I can more easily deprecate it (flutter/engine#8084)
7cbd9d8 Roll src/third_party/skia 3d1b941f3a7d..bea1f94f341e (7 commits) (flutter/engine#8083)
260669c Fix deleting text when the last character is some special characters on IOS (flutter/engine#7982)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
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.

3 participants