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

Conversation

@jason-simmons
Copy link
Member

The Dart SDK is now only building an AOT snapshot for the frontend server (see https://dart-review.googlesource.com/c/sdk/+/359100)

@jason-simmons jason-simmons requested a review from zanderso April 4, 2024 20:54
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

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

LGTM

@zanderso
Copy link
Member

zanderso commented Apr 4, 2024

cc @a-siva

@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 4, 2024
@auto-submit auto-submit bot merged commit 7feef43 into flutter:main Apr 4, 2024
Copy link
Contributor

@a-siva a-siva left a comment

Choose a reason for hiding this comment

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

lgtm

@a-siva a-siva requested a review from derekxu16 April 4, 2024 22:20
@a-siva
Copy link
Contributor

a-siva commented Apr 4, 2024

adding @derekxu16 to this PR as he was trying to land a similar PR

@zanderso
Copy link
Member

zanderso commented Apr 5, 2024

Reason for revert: This Dart roll is blocking the roll of the engine to the framework. Unblocking the rolls depends on addressing flutter/flutter#146164.

@zanderso zanderso added the revert Label used to revert changes in a closed and merged pull request. label Apr 5, 2024
auto-submit bot pushed a commit that referenced this pull request Apr 5, 2024
@auto-submit auto-submit bot removed the revert Label used to revert changes in a closed and merged pull request. label Apr 5, 2024
auto-submit bot added a commit that referenced this pull request Apr 5, 2024
…pshot filename (#51921)" (#51927)

Reverts: #51921
Initiated by: zanderso
Reason for reverting: This Dart roll is blocking the roll of the engine to the framework. Unblocking the rolls depends on addressing flutter/flutter#146164.
Original PR Author: jason-simmons

Reviewed By: {zanderso, jonahwilliams}

This change reverts the following previous change:
The Dart SDK is now only building an AOT snapshot for the frontend server (see https://dart-review.googlesource.com/c/sdk/+/359100)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 5, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 5, 2024
…146322)

flutter/engine@d44462a...f17d586

2024-04-05 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Manual Dart roll to 0ac840ba1 and update frontend server snapshot filename (#51921)" (flutter/engine#51927)
2024-04-05 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Roll Dart SDK from 0ac840ba1f0b to 31ddd6924103 (5 revisions) (#51924)" (flutter/engine#51926)
2024-04-05 [email protected] Roll Dart SDK from 0ac840ba1f0b to 31ddd6924103 (5 revisions) (flutter/engine#51924)
2024-04-04 [email protected] Manual Dart roll to 0ac840ba1 and update frontend server snapshot filename (flutter/engine#51921)

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 [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
gilnobrega pushed a commit to gilnobrega/flutter that referenced this pull request Apr 22, 2024
…lutter#146322)

flutter/engine@d44462a...f17d586

2024-04-05 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Manual Dart roll to 0ac840ba1 and update frontend server snapshot filename (flutter#51921)" (flutter/engine#51927)
2024-04-05 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Roll Dart SDK from 0ac840ba1f0b to 31ddd6924103 (5 revisions) (flutter#51924)" (flutter/engine#51926)
2024-04-05 [email protected] Roll Dart SDK from 0ac840ba1f0b to 31ddd6924103 (5 revisions) (flutter/engine#51924)
2024-04-04 [email protected] Manual Dart roll to 0ac840ba1 and update frontend server snapshot filename (flutter/engine#51921)

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 [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

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

Labels

autosubmit Merge PR when tree becomes green via auto submit App platform-fuchsia

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants