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

Conversation

@godofredoc
Copy link
Contributor

The artifacts have been used in prod for a few weeks already and this change is just migrating the generation of the artifacts to engine_v2 builds and moving the legacy recipe to staging.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The artifacts have been used in prod for a few weeks already and this
change is just migrating the generation of the artifacts to engine_v2
builds and moving the legacy recipe to staging.
@godofredoc godofredoc requested a review from zanderso April 17, 2023 22:06
@godofredoc
Copy link
Contributor Author

@zanderso can we land this one while we figure out the best way to remove the double zip of FlutterMacOS.framework.zip? flutter/flutter#124911

Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

lgtm, but can we wait to land infra changes until the tree recovers from flutter/flutter#125037 ?

@godofredoc
Copy link
Contributor Author

lgtm, but can we wait to land infra changes until the tree recovers from flutter/flutter#125037 ?

Thanks, seems like the tree has stabilized. Adding the autosubmit label.

@godofredoc godofredoc added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 18, 2023
@auto-submit auto-submit bot merged commit c42fbb1 into flutter:main Apr 18, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 18, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 18, 2023
…125069)

flutter/engine@6d263ea...5fcc7b7

2023-04-18 [email protected] [Impeller] Gpu model information to Skia gold (flutter/engine#41216)
2023-04-18 [email protected] [Impeller] faster glyph atlas generation by removing data copies (flutter/engine#41290)
2023-04-18 [email protected] Migrate android AOT to engine_v2. (flutter/engine#41229)
2023-04-18 [email protected] Roll Skia from 5bd4bdc0d8e2 to f80ee1088861 (8 revisions) (flutter/engine#41302)

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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@godofredoc godofredoc deleted the migrate_mac_aot branch June 5, 2023 23:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants