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

Conversation

@blasten
Copy link

@blasten blasten commented Aug 8, 2019

This is the first step to generate the POM files for the engine artifacts.

@matthew-carroll
Copy link
Contributor

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.

Copy link
Contributor

@mklim mklim left a 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"
}
[
{
Copy link
Contributor

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.

Copy link
Contributor

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.

@blasten blasten requested a review from dnfield August 8, 2019 23:49
Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

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

LGTM

engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 9, 2019
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.
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.

6 participants