-
Notifications
You must be signed in to change notification settings - Fork 6k
Include Maven dependency in files.json #10719
Conversation
|
Added @mklim due to his many reviews of embedding code, and work on the testing infrastructure for Android. I don't know this area really well, but LGTM. |
mklim
left a comment
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.
LGTM
| "android_support_compat.jar": "https://dl.google.com/dl/android/maven2/com/android/support/support-compat/28.0.0/support-compat-28.0.0.aar" | ||
| } | ||
| [ | ||
| { |
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.
FYI I think long term we wanted these moved to CIPD instead of here. I may be remembering wrong, I thought I saw an issue from @dnfield about this but couldn't find it now. Either way this is fine for this patch though.
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.
It would be preferable to create CIPD packages for these and avoid using this script any further.
dnfield
left a comment
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.
LGTM
git@github.com:flutter/engine.git/compare/dd735c917929...ff49ca1 git log dd735c9..ff49ca1 --no-merges --oneline 2019-08-09 dnfield@google.com Make firebase test more LUCI friendly (flutter/engine#10783) 2019-08-09 chinmaygarde@gmail.com Add a test for creating images from bytes. (flutter/engine#10799) 2019-08-09 skia-flutter-autoroll@skia.org Roll fuchsia/sdk/core/linux-amd64 from 1nsfv... to 9xVXx... (flutter/engine#10798) 2019-08-09 chinmaygarde@gmail.com Wire up a concurrent message loop backed SkExecutor for Skia. (flutter/engine#10788) 2019-08-09 stuartmorgan@google.com Roll buildroot for ANGLE support (flutter/engine#10784) 2019-08-09 chinmaygarde@gmail.com When setting up AOT snapshots from symbol references, make buffer sizes optional. (flutter/engine#10674) 2019-08-09 egarciad@google.com Include Maven dependency in files.json (flutter/engine#10719) 2019-08-09 liyuqian@google.com Remove 3 semi-redundant try-jobs (flutter/engine#10786) 2019-08-09 skia-flutter-autoroll@skia.org Roll src/third_party/skia 04580795746b..4d557e3df433 (4 commits) (flutter/engine#10794) 2019-08-09 jason-simmons@users.noreply.github.com Build JARs containing the Android embedding sources and the engine native library (flutter/engine#10778) 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 (jimgraham@google.com), and stop the roller if necessary.
This is the first step to generate the POM files for the engine artifacts.