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

Conversation

@godofredoc
Copy link
Contributor

@godofredoc godofredoc commented Apr 21, 2023

Reverts #41372

The duplicated artifacts were removed from Windows Host Engine in https://flutter-review.googlesource.com/c/recipes/+/42340. After the recipes change lands and is propagated it will be safe to update the Windows windows_arm_host_engine build to upload the artifacts to prod.

Bug: flutter/flutter#125262

@christopherfujino
Copy link
Contributor

christopherfujino commented Apr 21, 2023

What's the context for the re-land?

The initial change was reverted because both Windows Host Engine and Windows windows_arm_host_engine were trying to upload win arm64 artifacts to prod.

Removing the arm64 artifacts from Windows Host Engine(https://flutter-review.googlesource.com/c/recipes/+/42340.) will allow to reland the change.

@zanderso zanderso changed the title Revert "Revert "Upload windows arm artifacts to production bucket."" Reland "Upload windows arm artifacts to production bucket." Apr 24, 2023
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@godofredoc godofredoc added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 24, 2023
@auto-submit auto-submit bot merged commit d65def9 into main Apr 24, 2023
@auto-submit auto-submit bot deleted the revert-41372-revert-41324-move_win_arm_prod branch April 24, 2023 21:17
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 25, 2023
…125452)

flutter/engine@5fbde6c...8fe8e94

2023-04-25 [email protected] Revert "[impeller] convert src over to src for solid color" (flutter/engine#41466)
2023-04-24 [email protected] Remove empty tests properties. (flutter/engine#41461)
2023-04-24 [email protected] Roll Skia from 0ec4c84abd0b to 5f4dd36a69f9 (21 revisions) (flutter/engine#41462)
2023-04-24 [email protected] Reland "Upload windows arm artifacts to production bucket." (flutter/engine#41409)
2023-04-24 [email protected] Roll Dart SDK from c7042dc03f43 to c3afc3507b4f (1 revision) (flutter/engine#41458)
2023-04-24 [email protected] Add 16.2 runtime to Mac unopt (flutter/engine#41456)
2023-04-24 [email protected] Roll Fuchsia Mac SDK from EEdHfWcEn3fHwQj46... to gxpKJcSBfY7AuR_Ur... (flutter/engine#41453)
2023-04-24 [email protected] Roll Fuchsia Linux SDK from fJ7ysM-O4VYAGeBDc... to 8TJtGsJTpIHk5-Ss5... (flutter/engine#41451)
2023-04-24 [email protected] Roll Dart SDK from a1c1e0d3bb18 to c7042dc03f43 (2 revisions) (flutter/engine#41452)
2023-04-24 [email protected] Roll Skia from 1f84e5060646 to 0ec4c84abd0b (6 revisions) (flutter/engine#41449)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from fJ7ysM-O4VYA to 8TJtGsJTpIHk
  fuchsia/sdk/core/mac-amd64 from EEdHfWcEn3fH to gxpKJcSBfY7A

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
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