| commit | 63c8af00d0f611579b52fa1192c937c57aaf1962 | [log] [tgz] |
|---|---|---|
| author | Devon Loehr <dloehr@google.com> | Fri May 30 21:05:03 2025 |
| committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Fri May 30 21:05:03 2025 |
| tree | 108fc6f4eac1e030d38e291297c341a07fa45702 | |
| parent | 314addb81670c24736152d2685b288fcd717840e [diff] |
Fail build if clang emits any warnings This inserts a check of clang's output to see if it emitted any warnings during the compilation. Normally, warnings will cause immediate failure, but if without -Werror, they won't. In that case, we still want to mark that something is wrong, so report a failure. It would be preferable to report a WARNING status, but this doesn't seem to be supported at the moment: crbug.com/40581344 Bug: 419005149 Change-Id: Ia921c6900326e618b7618b0685529a5c9000cbfc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/6580258 Reviewed-by: Fumitoshi Ukai <ukai@google.com> Reviewed-by: Ben Pastene <bpastene@chromium.org> Commit-Queue: Devon Loehr <dloehr@google.com>
If you're here to make a change to ‘recipes’ (the code located in the recipes directory), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent. Functions use lowercase_with_underscores, with the exception of the special functions RunSteps and RunTests in recipes. Use yapf (git cl format --no-clang-format) to autoformat new code.