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

Reverts "Remove Fuchsia Mac SDK from DEPS" #47791

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

auto-submit[bot]
Copy link
Contributor

@auto-submit auto-submit bot commented Nov 8, 2023

Reverts #47700
Initiated by: zanderso
This change reverts the following previous change:
Original Description:
Save infra capacity. Fuchsia commits 4 times a day, which can trigger a subsequent framework roll. Removing the Mac roller as it appears to be unused anymore.

Skia config is in https://skia-review.googlesource.com/c/skia-autoroll-internal-config/+/774657

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

@auto-submit auto-submit bot added the revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests. label Nov 8, 2023
@zanderso zanderso requested a review from CaseyHillers November 8, 2023 04:31
@auto-submit auto-submit bot merged commit 9250de9 into main Nov 8, 2023
@zanderso
Copy link
Member

zanderso commented Nov 8, 2023

The Engine -> Framework autoroller is looking for this:

Failed to transition from "success" to "idle": failed to get next revision to roll from Child: Unable to find "fuchsia/sdk/core/mac-amd64" in DEPS! Entries:

@auto-submit auto-submit bot deleted the revert_a1bad0749afc20318deb5394e7b198943d204b48 branch November 8, 2023 04:32
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 8, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 8, 2023
…138091)

flutter/engine@3e3be5e...117d47a

2023-11-08 [email protected] Roll Skia from fce71a80b0a2 to a4cce5236dcf (1 revision) (flutter/engine#47807)
2023-11-08 [email protected] [macOS] Clean up resources in ViewController tests (flutter/engine#47792)
2023-11-08 [email protected] Roll Skia from f3d250126ba9 to fce71a80b0a2 (1 revision) (flutter/engine#47796)
2023-11-08 [email protected] Roll Skia from b4fa927468e6 to f3d250126ba9 (1 revision) (flutter/engine#47793)
2023-11-08 [email protected] [Impeller] Add Rect::GetNormalizingTransform to handle UV coordinate conversion (flutter/engine#47775)
2023-11-08 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Remove Fuchsia Mac SDK from DEPS" (flutter/engine#47791)
2023-11-08 [email protected] [web] fix clicks on merged semantic nodes (attempt #2) (flutter/engine#47360)
2023-11-08 [email protected] Roll Skia from 0f78e5f765d3 to b4fa927468e6 (1 revision) (flutter/engine#47788)
2023-11-08 [email protected] Roll Fuchsia Linux SDK from VcFEJiUUTYwkhEAlJ... to sD8HRA4JmXczujkqO... (flutter/engine#47785)
2023-11-08 [email protected] Fix narrowing conversion lint (flutter/engine#47740)
2023-11-08 [email protected] [macOS] Bail out of tests if engine not running (flutter/engine#47771)
2023-11-08 [email protected] Roll Skia from f91d39395e85 to 0f78e5f765d3 (1 revision) (flutter/engine#47776)
2023-11-08 [email protected] [testing] Extract StreamCapture test utility (flutter/engine#47774)
2023-11-08 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Promote fuchsia build v2 to prod." (flutter/engine#47779)
2023-11-08 [email protected] Include updated locations for dart third_party components into license ignore-list. (flutter/engine#47770)
2023-11-07 [email protected] Remove Fuchsia Mac SDK from DEPS (flutter/engine#47700)
2023-11-07 [email protected] Roll Skia from 030e21befbc9 to f91d39395e85 (6 revisions) (flutter/engine#47769)
2023-11-07 [email protected] [Impeller] Support static thread safety analysis with condition variables. (flutter/engine#47763)
2023-11-07 [email protected] Promote fuchsia build v2 to prod. (flutter/engine#47729)
2023-11-07 [email protected] [Windows] Reduce warnings produced by unit tests (flutter/engine#47724)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from VcFEJiUUTYwk to sD8HRA4JmXcz

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://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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants