| commit | 55624ef304daebb5b15ff83856d8b62067c1dedf | [log] [tgz] |
|---|---|---|
| author | Garrett Beaty <gbeaty@chromium.org> | Mon May 11 18:22:54 2020 |
| committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Mon May 11 18:22:54 2020 |
| tree | d09f36c4daadae20a44c4700181bc3d268c52295 | |
| parent | 40025992495ac6d5dce0ae5debf0caa6f133471f [diff] |
Change chromium.compile behavior to not pass unnecessary targets. If there is a non-empty list of targets to compile that includes the target 'all', then it is equivalent to not passing any targets on the command line. Changing this behavior will allow for removing the very specific configuration option add_tests_as_compile_targets from chromium_tests.BotSpec. Bug: 1080768 Change-Id: I3569f66abd26d5f3e1387e2ab4916295d9dfb489 Recipe-Nontrivial-Roll: build_limited_scripts_slave Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2190994 Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by: Aaron Gable <agable@chromium.org> Reviewed-by: Stephen Martinis <martiniss@chromium.org>
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style. Functions use lowercase_with_underscores, with the exception of the special functions RunSteps and RunTests in recipes. Use yapf (git cl format) to autoformat new code.