-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Copy the flutter_gpu package into the engine artifacts. #51365
Copy the flutter_gpu package into the engine artifacts. #51365
Conversation
@christopherfujino Will adding new artifacts here confuse any tests in the framework repo? The code signing tests? |
Yeah, if this is a binary then I think we need to add a rule to codesign it, I think here: https://github.com/flutter/flutter/wiki/code-signing-metadata#add--update-code-signing-metadata If these are already codesigned, then this should be fine. |
This is purely a Dart package and doesn't contain any native binaries. So I think we're good? |
Gonna give this a land and see how it shakes out in the roll. |
…145062) flutter/engine@ea848c3...285b9fb 2024-03-13 bdero@google.com Copy the flutter_gpu package into the engine artifacts. (flutter/engine#51365) 2024-03-13 chinmaygarde@google.com [Impeller] Make masks type safe. (flutter/engine#51369) 2024-03-12 bdero@google.com Remove early Flutter GPU prototype from ui/experiments. (flutter/engine#51367) 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 bdero@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
Part of flutter/flutter#131711.