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

Conversation

@sjindel-google
Copy link
Contributor

No description provided.

Copy link
Contributor

@alexmarkov alexmarkov left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -1,46 +1 @@
dart:_internal,::,_printClosure
Copy link
Contributor

Choose a reason for hiding this comment

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

You need to wait until your change in gen_snapshot is rolled into Flutter (or roll Dart before landing this PR).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep.

@sjindel-google sjindel-google merged commit 9e315e6 into flutter:master Sep 18, 2018
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 18, 2018
flutter/engine@dcbc8a8...9e315e6

git log dcbc8a8..9e315e6 --no-merges --oneline
9e315e6 Remove entries from dart_vm_entry_points.txt and use annotations (flutter/engine#6226)
cdbe214 Update test .gitignore (flutter/engine#6272)
dbfa827 [fuchsia] Fixes for the Fuchsia build (flutter/engine#6273)

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.
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Sep 18, 2018
flutter/engine@dcbc8a8...9e315e6

git log dcbc8a8..9e315e6 --no-merges --oneline
9e315e6 Remove entries from dart_vm_entry_points.txt and use annotations (flutter/engine#6226)
cdbe214 Update test .gitignore (flutter/engine#6272)
dbfa827 [fuchsia] Fixes for the Fuchsia build (flutter/engine#6273)


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.
amirh pushed a commit to amirh/engine that referenced this pull request Sep 21, 2018
…tter#6226)

The `dart_vm_entry_points.txt` file is now deprecated in favor of annotations.
Support for parsing and generating the entry-points files will be removed from Dart shortly.
amirh pushed a commit to amirh/engine that referenced this pull request Sep 21, 2018
…tter#6226)

The `dart_vm_entry_points.txt` file is now deprecated in favor of annotations.
Support for parsing and generating the entry-points files will be removed from Dart shortly.
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.

3 participants