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

Remove --enable-impeller-{backend} flags (always True). - #51402

Merged
matanlurey merged 5 commits into
flutter-team-archive:mainfrom
matanlurey:remove-default-true-impeller-be-flags
Mar 14, 2024
Merged

matanlurey merged 5 commits into
flutter-team-archive:mainfrom
matanlurey:remove-default-true-impeller-be-flags

Conversation

@matanlurey

Copy link
Copy Markdown
Contributor

@jonahwilliams

Copy link
Copy Markdown
Contributor

Something seems up with CI

@matanlurey

Copy link
Copy Markdown
Contributor Author

I guess I broke the YAML format. Will fix it tomorrow 🙂

@matanlurey

Copy link
Copy Markdown
Contributor Author

I guess I broke the YAML format. Will fix it tomorrow 🙂

Or not, it was easy

@matanlurey matanlurey changed the title Remove --enable-impeller-{backend} flags that default to True. Remove --enable-impeller-{backend} flags (always True). Mar 14, 2024

@jonahwilliams jonahwilliams left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@matanlurey
matanlurey merged commit 1da1597 into flutter-team-archive:main Mar 14, 2024
@matanlurey
matanlurey deleted the remove-default-true-impeller-be-flags branch March 14, 2024 05:13
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Mar 14, 2024
flutter-team-archive/engine@bdf66af...1da1597

2024-03-14 [email protected] Remove `--enable-impeller-{backend}` flags (always `True`). (flutter-team-archive/engine#51402)

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
@chinmaygarde

Copy link
Copy Markdown
Contributor

Awesome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Development

Successfully merging this pull request may close these issues.

[Impeller] delete --enable-impeller-vulkan and --enable-impeller-opengles

3 participants