-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Roll buildroot to pick up iOS 12 #49542
Conversation
@jason-simmons this PR has the failure that needs the fix to the sed command I mentioned here flutter/buildroot#806 (review) |
d8f7516
to
ae4879a
Compare
@jason-simmons this roll has
|
b5cdbad
to
a530f35
Compare
@jmagman I edited this PR to pick up another patch from @jason-simmons and rebased. |
Ohh, the change to the sed flag is somehow leaking into the clang-tidy command?
|
Yes - confirmed locally that the Working on a patch that will filter out the sed command when clang-tidy builds |
Why aren't these showing up during
Is the problem that |
This is happening because the engine's clang-tidy wrapper script is parsing The clang-tidy wrapper is not expecting the |
…y be appended to each compile command See flutter#49542
…y be appended to each compile command See flutter#49542
…y be appended to each compile command See flutter#49542
3939a00
to
bd0f5a1
Compare
…141152) flutter/engine@d134799...820645d 2024-01-09 skia-flutter-autoroll@skia.org Roll Skia from e3af4a11007d to 06404ed554e7 (1 revision) (flutter/engine#49614) 2024-01-09 magder@google.com Roll buildroot to pick up iOS 12 (flutter/engine#49542) 2024-01-09 skia-flutter-autoroll@skia.org Roll Skia from f4626a2e3293 to e3af4a11007d (1 revision) (flutter/engine#49613) 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 rmistry@google.com,zra@google.com 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
flutter/buildroot#804
flutter/buildroot#806
flutter/buildroot#808
flutter/buildroot#809
flutter/buildroot#810
Engine tests were already running on iOS 13.
Part of flutter/flutter#140474
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.