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

Roll Clang from 00396e6a1a0b to b3a9e8f7c0af#48563

Closed
skia-flutter-autoroll wants to merge 1 commit into
flutter-team-archive:mainfrom
skia-flutter-autoroll:clang-flutter-engine-8b57dbd5-0233-4fb1-96f5-ad7fe100bbbf-1701421092
Closed

Roll Clang from 00396e6a1a0b to b3a9e8f7c0af#48563
skia-flutter-autoroll wants to merge 1 commit into
flutter-team-archive:mainfrom
skia-flutter-autoroll:clang-flutter-engine-8b57dbd5-0233-4fb1-96f5-ad7fe100bbbf-1701421092

Conversation

@skia-flutter-autoroll

Copy link
Copy Markdown
Contributor

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/clang-flutter-engine
Please CC matanl@google.com,rmistry@google.com,zanderso@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Clang: https://bugs.fuchsia.dev/p/fuchsia/issues/list?q=component%3AToolchain
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

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/clang-flutter-engine
Please CC matanl@google.com,rmistry@google.com,zanderso@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Clang: https://bugs.fuchsia.dev/p/fuchsia/issues/list?q=component%3AToolchain
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
@skia-flutter-autoroll skia-flutter-autoroll added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@skia-flutter-autoroll skia-flutter-autoroll force-pushed the clang-flutter-engine-8b57dbd5-0233-4fb1-96f5-ad7fe100bbbf-1701421092 branch from 981f089 to 57f4e61 Compare December 1, 2023 08:58
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@auto-submit

auto-submit Bot commented Dec 1, 2023

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/48563, due to - The status or check suite Windows windows_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

  • The status or check suite Mac mac_clang_tidy_presubmit has failed. Please fix the issues identified (or deflake) before re-applying this label.

@skia-flutter-autoroll skia-flutter-autoroll added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@auto-submit

auto-submit Bot commented Dec 1, 2023

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/48563, due to - The status or check suite Windows windows_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

  • The status or check suite Windows windows_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_ios_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Linux mac_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_clang_tidy_presubmit has failed. Please fix the issues identified (or deflake) before re-applying this label.

@skia-flutter-autoroll skia-flutter-autoroll added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@auto-submit

auto-submit Bot commented Dec 1, 2023

Copy link
Copy Markdown
Contributor

auto label is removed for flutter/engine/48563, due to - The status or check suite Windows windows_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

  • The status or check suite Windows windows_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_ios_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Linux mac_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_host_engine has failed. Please fix the issues identified (or deflake) before re-applying this label.
  • The status or check suite Mac mac_clang_tidy_presubmit has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 1, 2023
@skia-flutter-autoroll

Copy link
Copy Markdown
Contributor Author

Commit queue failed; closing this roll.

chinmaygarde added a commit to chinmaygarde/flutter_engine that referenced this pull request Dec 5, 2023
I was shocked this was even legal earlier. But a new Clang roll makes
this an a warning which gets converted into an error.

I attempted to roll to the Clang roll the roller did and fixed the
issues I found. Hopefully, the next roll is unblocked.

Original failure: flutter-team-archive#48563
auto-submit Bot pushed a commit that referenced this pull request Dec 5, 2023
I was shocked this was even legal earlier. But a new Clang roll makes this an a warning which gets converted into an error.

I attempted to roll to the Clang roll the roller did and fixed the issues I found. Hopefully, the next roll is unblocked.

Original failure: #48563
chinmaygarde added a commit to chinmaygarde/flutter_engine that referenced this pull request Dec 5, 2023
auto-submit Bot pushed a commit that referenced this pull request Dec 7, 2023
Previously attempted by #48563 but failed because of fixes needed in #48693
auto-submit Bot added a commit that referenced this pull request Dec 7, 2023
#48802)

Reverts #48705
Initiated by: zanderso
This change reverts the following previous change:
Original Description:
Previously attempted by #48563 but failed because of fixes needed in #48693
@skia-flutter-autoroll skia-flutter-autoroll deleted the clang-flutter-engine-8b57dbd5-0233-4fb1-96f5-ad7fe100bbbf-1701421092 branch December 8, 2023 09:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants