Skip to content
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

Merged
merged 11 commits into from
Aug 9, 2019
Merged

Include Maven dependency in files.json #10719

merged 11 commits into from
Aug 9, 2019

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

[
{
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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants