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

Reduce rebuild times when invoking 'et run' #52883

Merged
merged 1 commit into from
May 22, 2024

Conversation

johnmccutchan
Copy link
Contributor

Android only for now but this avoids rebuilding a bunch of test targets that aren't necessary to do a flutter run with a custom engine.

I need to fill in some blanks for other platforms before landing.

@johnmccutchan
Copy link
Contributor Author

Build time for a single line change to Android Engine

before: 76 seconds
after: 41 seconds

1.85x speedup

tools/engine_tool/lib/src/commands/run_command.dart Outdated Show resolved Hide resolved
tools/engine_tool/lib/src/gn.dart Outdated Show resolved Hide resolved
tools/engine_tool/lib/src/run_utils.dart Show resolved Hide resolved
tools/engine_tool/test/run_command_test.dart Outdated Show resolved Hide resolved
Copy link
Contributor

@matanlurey matanlurey left a comment

Choose a reason for hiding this comment

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

Address Zach's comments but otherwise LGTM.

tools/engine_tool/lib/src/commands/run_command.dart Outdated Show resolved Hide resolved
Android only for now but this avoids rebuilding a bunch of test targets
that aren't necessary to do a `flutter run` with a custom engine.
@johnmccutchan johnmccutchan added the autosubmit Merge PR when tree becomes green via auto submit App label May 22, 2024
@auto-submit auto-submit bot merged commit 6f73bd9 into flutter:main May 22, 2024
29 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 22, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 22, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 22, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 22, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request May 23, 2024
…148919)

flutter/engine@8b094fb...b8b8245

2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 380ae98608de to 0ee149894a8d (5 revisions) (flutter/engine#52989)
2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 396c66076ab1 to 380ae98608de (4 revisions) (flutter/engine#52983)
2024-05-22 john@johnmccutchan.com Reduce rebuild times when invoking 'et run' (flutter/engine#52883)
2024-05-22 1961493+harryterkelsen@users.noreply.github.com [canvaskit] Add dilate and erode imagefilters (flutter/engine#48553)
2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 93ed643df6d1 to 396c66076ab1 (3 revisions) (flutter/engine#52981)
2024-05-22 john@johnmccutchan.com Fix another instance of platform view breakage on Android 14 (flutter/engine#52980)

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 jonahwilliams@google.com,rmistry@google.com,zra@google.com 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
victorsanni pushed a commit to victorsanni/flutter that referenced this pull request May 31, 2024
…lutter#148919)

flutter/engine@8b094fb...b8b8245

2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 380ae98608de to 0ee149894a8d (5 revisions) (flutter/engine#52989)
2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 396c66076ab1 to 380ae98608de (4 revisions) (flutter/engine#52983)
2024-05-22 john@johnmccutchan.com Reduce rebuild times when invoking 'et run' (flutter/engine#52883)
2024-05-22 1961493+harryterkelsen@users.noreply.github.com [canvaskit] Add dilate and erode imagefilters (flutter/engine#48553)
2024-05-22 skia-flutter-autoroll@skia.org Roll Skia from 93ed643df6d1 to 396c66076ab1 (3 revisions) (flutter/engine#52981)
2024-05-22 john@johnmccutchan.com Fix another instance of platform view breakage on Android 14 (flutter/engine#52980)

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 jonahwilliams@google.com,rmistry@google.com,zra@google.com 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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
3 participants