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

Roll Skia from f7e3a5395fe1 to a1c58c4eb6d5 (2 revisions)#50609

Closed
skia-flutter-autoroll wants to merge 1 commit into
flutter-team-archive:mainfrom
skia-flutter-autoroll:skia-flutter-autoroll-038f6d3d-cc85-4f43-99b9-7513259cd1d7-1707855217
Closed

Roll Skia from f7e3a5395fe1 to a1c58c4eb6d5 (2 revisions)#50609
skia-flutter-autoroll wants to merge 1 commit into
flutter-team-archive:mainfrom
skia-flutter-autoroll:skia-flutter-autoroll-038f6d3d-cc85-4f43-99b9-7513259cd1d7-1707855217

Conversation

@skia-flutter-autoroll

Copy link
Copy Markdown
Contributor

https://skia.googlesource.com/skia.git/+log/f7e3a5395fe1..a1c58c4eb6d5

2024-02-13 weiyuhuang@google.com Fix Half Leading for Strut.
2024-02-13 kjlubick@google.com Define GR_TEST_UTILS when building skqp for Android Framework

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,johnstiles@google.com,jsimmons@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
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

https://skia.googlesource.com/skia.git/+log/f7e3a5395fe1..a1c58c4eb6d5

2024-02-13 weiyuhuang@google.com Fix Half Leading for Strut.
2024-02-13 kjlubick@google.com Define GR_TEST_UTILS when building skqp for Android Framework

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-flutter-autoroll
Please CC brianosman@google.com,johnstiles@google.com,jsimmons@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Skia: https://bugs.chromium.org/p/skia/issues/entry
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
@skia-flutter-autoroll
skia-flutter-autoroll force-pushed the skia-flutter-autoroll-038f6d3d-cc85-4f43-99b9-7513259cd1d7-1707855217 branch from 981f089 to 20d6d8a Compare February 13, 2024 20:13
@skia-flutter-autoroll skia-flutter-autoroll added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 13, 2024
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Feb 13, 2024
@auto-submit

auto-submit Bot commented Feb 13, 2024

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/50609, due to - The status or check suite Linux linux_web_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.

@skia-flutter-autoroll

Copy link
Copy Markdown
Contributor Author

Commit queue failed; closing this roll.

auto-submit Bot pushed a commit that referenced this pull request Feb 14, 2024
#50617)

The previous skia roll (#50609) failed on [Linux linux_web_engine](https://ci.chromium.org/b/8756177705618610401).
I've tested that the failing `canvaskit_api_tt_on_test.dart` would pass with the new skia revision `'a1c58c4eb6d5ae553097c5d27c134e09c8825e06` on this patch.

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
@skia-flutter-autoroll
skia-flutter-autoroll deleted the skia-flutter-autoroll-038f6d3d-cc85-4f43-99b9-7513259cd1d7-1707855217 branch February 20, 2024 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants