-
Notifications
You must be signed in to change notification settings - Fork 6k
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.