-
Notifications
You must be signed in to change notification settings - Fork 6k
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
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. |
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
[ | ||
{ |
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.
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
[email protected]:flutter/engine.git/compare/dd735c917929...ff49ca1 git log dd735c9..ff49ca1 --no-merges --oneline 2019-08-09 [email protected] Make firebase test more LUCI friendly (flutter/engine#10783) 2019-08-09 [email protected] Add a test for creating images from bytes. (flutter/engine#10799) 2019-08-09 [email protected] Roll fuchsia/sdk/core/linux-amd64 from 1nsfv... to 9xVXx... (flutter/engine#10798) 2019-08-09 [email protected] Wire up a concurrent message loop backed SkExecutor for Skia. (flutter/engine#10788) 2019-08-09 [email protected] Roll buildroot for ANGLE support (flutter/engine#10784) 2019-08-09 [email protected] When setting up AOT snapshots from symbol references, make buffer sizes optional. (flutter/engine#10674) 2019-08-09 [email protected] Include Maven dependency in files.json (flutter/engine#10719) 2019-08-09 [email protected] Remove 3 semi-redundant try-jobs (flutter/engine#10786) 2019-08-09 [email protected] Roll src/third_party/skia 04580795746b..4d557e3df433 (4 commits) (flutter/engine#10794) 2019-08-09 [email protected] 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 ([email protected]), and stop the roller if necessary.
This is the first step to generate the POM files for the engine artifacts.