Skip to content

Conversation

@zanderso
Copy link
Member

@zanderso zanderso commented Nov 7, 2021

Reverts #93117

Really sorry about this, but the Engine roll is 40+ commits behind, and other failures are being masked by this one. I'm going to start getting the trees untangled by rolling this back along with the companion change in the Engine: flutter/engine#29574

@flutter-dashboard flutter-dashboard bot added a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation engine flutter/engine related. See also e: labels. f: integration_test The flutter/packages/integration_test plugin c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Nov 7, 2021
@google-cla google-cla bot added the cla: yes label Nov 7, 2021
@zanderso zanderso merged commit 2468f31 into master Nov 7, 2021
@zanderso zanderso deleted the revert-93117-reland_engine_display_features branch November 7, 2021 05:46
@zanderso
Copy link
Member Author

zanderso commented Nov 8, 2021

/cc @blasten @andreidiaconu The Engine roll is super backed up on #93226 and #93225. Please hold off on re-landing this work until the Engine backlog has cleared. Sorry for the inconvenience!

@andreidiaconu
Copy link
Contributor

andreidiaconu commented Nov 8, 2021

Third time's a charm 👍. Please let me know if I can help in any way.

andreidiaconu added a commit to andreidiaconu/flutter that referenced this pull request Nov 8, 2021
@andreidiaconu
Copy link
Contributor

/cc @blasten @andreidiaconu The Engine roll is super backed up on #93226 and #93225. Please hold off on re-landing this work until the Engine backlog has cleared. Sorry for the inconvenience!

#93226 is closed and #93225 is no longer a P0. Can we try to reland this?

@zanderso
Copy link
Member Author

I should have noted this originally, but the remaining failure on CI that led to the reverts was just this one:

https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android%20flutter_engine_group_performance/912/overview

If you know how to keep that one from failing again, then I think you can try to reland.

@andreidiaconu
Copy link
Contributor

Ok, I fixed that one in #93240 and I think we are ready to try and reland.

WizzXu pushed a commit to WizzXu/flutter that referenced this pull request Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos engine flutter/engine related. See also e: labels. f: integration_test The flutter/packages/integration_test plugin tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants