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

Remove cores dimension from Mac Host clang-tidy #41214

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

jmagman
Copy link
Member

@jmagman jmagman commented Apr 14, 2023

Follow up to #41183. Remove the cores dimensions so this builder can run on any arm machine which currently all have 8 cores but there's no reason to specify now that the arch is arm.

Introduced in #38261 to avoid 4-core Intel machines.

@jmagman jmagman self-assigned this Apr 14, 2023
@jmagman jmagman marked this pull request as ready for review April 14, 2023 20:45
@jmagman jmagman requested review from zanderso and keyonghan April 14, 2023 20:46
Copy link
Contributor

@keyonghan keyonghan left a comment

Choose a reason for hiding this comment

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

LGTM

@jmagman jmagman added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 14, 2023
@auto-submit auto-submit bot merged commit 7bbc8be into flutter:main Apr 14, 2023
@jmagman jmagman deleted the cores branch April 14, 2023 21:07
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 14, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Apr 15, 2023
…124900)

flutter/engine@e9739bc...f794529

2023-04-14 [email protected] Change v2 `mac_unopt` orchestrator builder to Linux (flutter/engine#41184)
2023-04-14 [email protected] Specify allowed Macmini models (flutter/engine#41219)
2023-04-14 [email protected] Migrate mac_host_engine to engine v2 builds. (flutter/engine#41149)
2023-04-14 [email protected] Remove cores dimension from Mac Host clang-tidy (flutter/engine#41214)
2023-04-14 [email protected] Run mac_host_engine orchestrator on arm Macs (flutter/engine#41185)

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
exaby73 pushed a commit to NevercodeHQ/flutter that referenced this pull request Apr 17, 2023
…lutter#124900)

flutter/engine@e9739bc...f794529

2023-04-14 [email protected] Change v2 `mac_unopt` orchestrator builder to Linux (flutter/engine#41184)
2023-04-14 [email protected] Specify allowed Macmini models (flutter/engine#41219)
2023-04-14 [email protected] Migrate mac_host_engine to engine v2 builds. (flutter/engine#41149)
2023-04-14 [email protected] Remove cores dimension from Mac Host clang-tidy (flutter/engine#41214)
2023-04-14 [email protected] Run mac_host_engine orchestrator on arm Macs (flutter/engine#41185)

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.

3 participants