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

Conversation

@mklim
Copy link
Contributor

@mklim mklim commented Aug 27, 2019

Fixes flutter/flutter#39284.

Actually guaranteeing the right binary for a given version is OS looks intricate to guarantee across arbitrary installs, operating systems, and linux distros. I think this should be enough notice for now to prevent any future confusion from accidentally running the tests on the wrong version.

Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

@mklim mklim added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Aug 27, 2019
@mklim mklim merged commit 0287144 into flutter:master Aug 27, 2019
@mklim mklim deleted the java_test_version branch August 27, 2019 17:38
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 27, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 27, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 28, 2019
git@github.com:flutter/engine.git/compare/bf7796611dc1...f93b9cb

git log bf77966..f93b9cb --no-merges --oneline
2019-08-27 bkonyi@google.com Roll src/third_party/dart 4015a79210..f17d0563fa (5 commits)
2019-08-27 dnfield@google.com Optionally strip bitcode when creating ios framework (flutter/engine#11530)
2019-08-27 dnfield@google.com Bitcode only for release (flutter/engine#11520)
2019-08-27 dnfield@google.com Strip bitcode from gen_snapshot (flutter/engine#11528)
2019-08-27 chris@bracken.jp Reuse texture cache in ios_external_texture_gl (flutter/engine#11524)
2019-08-27 skia-flutter-autoroll@skia.org Roll src/third_party/skia 4b5d65561943..6064ecf7e993 (1 commits) (flutter/engine#11500)
2019-08-27 mklim@google.com Assert that the JUnit tests are running on Java 8 (flutter/engine#11466)
2019-08-27 bkonyi@google.com Roll src/third_party/dart e878bdafe9..4015a79210 (15 commits)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC garyq@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

run_tests.py should enforce a java version for the JUnit tests

4 participants