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

Use application_kernel_asset in flutter_tester #6972

Merged
merged 1 commit into from
Nov 27, 2018

Conversation

cbracken
Copy link
Member

Since the migration to Dart 2, the first positional command-line
argument to flutter_tester has been a Dart kernel .dill file. Use
Settings::application_kernel_asset here for consistency/clarity.

Since the migration to Dart 2, the first positional command-line
argument to flutter_tester has been a Dart kernel .dill file. Use
Settings::application_kernel_asset here for consistency/clarity.
@cbracken cbracken merged commit 2ab0194 into flutter:master Nov 27, 2018
@cbracken cbracken deleted the use-application-kernel-path branch November 27, 2018 21:51
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 27, 2018
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Nov 28, 2018
…e#6972) (#24803)

flutter/engine@5bf4deb...2ab0194

git log 5bf4deb..2ab0194 --no-merges --oneline
2ab0194 Use application_kernel_asset in flutter_tester (flutter/engine#6972)


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, who should
be CC'd on the roll, and stop the roller if necessary.
GaryQian pushed a commit to GaryQian/engine that referenced this pull request Dec 11, 2018
Since the migration to Dart 2, the first positional command-line
argument to flutter_tester has been a Dart kernel .dill file. Use
Settings::application_kernel_asset here for consistency/clarity.
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.

3 participants