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

Include Maven dependency in files.json#10719

Merged
blasten merged 11 commits intoflutter:masterfrom
blasten:include_maven_dep
Aug 9, 2019
Merged

Include Maven dependency in files.json#10719
blasten merged 11 commits intoflutter:masterfrom
blasten:include_maven_dep

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"
} No newline at end of file
[
{
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
[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.
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