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

Conversation

@keyonghan
Copy link
Contributor

@keyonghan keyonghan commented Apr 20, 2023

@keyonghan
Copy link
Contributor Author

Filed flutter/flutter#125216 for Linux mac_unopt failure.

Comment on lines 65 to 67
"runtime_versions": [
"ios-16-0_14a5294e"
],
Copy link
Member

Choose a reason for hiding this comment

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

I believe the simulator runtimes need to be updated as well:
flutter/flutter#125216 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added and the CI is happy now.

Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

Temporarily blocking this, @zanderso has requested we hold off on this (and the inevitable slow down populating caches) until the very delayed engine -> framework roll gets unwedged. flutter/flutter#125221

This wouldn't directly impact the roll, but if there are changes needed in the engine to get it passing in the framework then engine build delays would slow it down.

@zanderso can you let us know when this is safe to land?

Copy link
Member

@jmagman jmagman left a comment

Choose a reason for hiding this comment

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

Looks like the roll succeeded flutter/flutter#125271
LGTM!

@keyonghan keyonghan added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 21, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 21, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Apr 21, 2023

auto label is removed for flutter/engine, pr: 41370, due to This PR has not met approval requirements for merging. Changes were requested by {jmagman}, please make the needed changes and resubmit this PR.
You have project association MEMBER and need 1 more review(s) in order to merge this PR.

  • Merge guidelines: You need at least one approved review if you are already a MEMBER or two member reviews if you are not a MEMBER before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

@keyonghan
Copy link
Contributor Author

auto label is removed for flutter/engine, pr: 41370, due to This PR has not met approval requirements for merging. Changes were requested by {jmagman}, please make the needed changes and resubmit this PR. You have project association MEMBER and need 1 more review(s) in order to merge this PR.

  • Merge guidelines: You need at least one approved review if you are already a MEMBER or two member reviews if you are not a MEMBER before re-applying the autosubmit label. Reviewers: If you left a comment approving, please use the "approve" review action instead.

Seems a bug with auto submit bot. created flutter/flutter#125280 to track. Manually submit here.

@keyonghan keyonghan merged commit 267f248 into flutter:main Apr 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 21, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 21, 2023
zanderso pushed a commit to flutter/flutter that referenced this pull request Apr 21, 2023
…125290)

flutter/engine@122c3b3...a7899c4

2023-04-21 [email protected] Roll Skia from 524ecb58945e to
90a93bb0990f (1 revision) (flutter/engine#41390)
2023-04-21 [email protected] Roll Fuchsia Mac SDK from
4OrPF9lzqCKGwBLRh... to fMOcI5zn1z0nOrV7s... (flutter/engine#41389)
2023-04-21 [email protected] Roll Skia from f4609aa2eaba to
524ecb58945e (23 revisions) (flutter/engine#41388)
2023-04-21 [email protected] Roll Dart SDK from
50b96abe9f6f to 4cc2e637a315 (1 revision) (flutter/engine#41387)
2023-04-21 [email protected] Update xcode to
14c18 (flutter/engine#41370)
2023-04-21 [email protected] Revert "[Impeller] Turned
on wide gamut support by default." (flutter/engine#41382)
2023-04-21 [email protected] Revert "Manual roll skia to d5b4acfb4"
(flutter/engine#41381)

Also rolling transitive DEPS:
  fuchsia/sdk/core/mac-amd64 from 4OrPF9lzqCKG to fMOcI5zn1z0n

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
@keyonghan keyonghan deleted the xcode_14c18 branch August 31, 2023 20:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The mac_unopt ios_debug_sim test fails with xcode 14c18

2 participants