| commit | 3da3be0aed7a29280f871ad9a136fa18a6444c02 | [log] [tgz] |
|---|---|---|
| author | Garrett Beaty <gbeaty@chromium.org> | Tue Sep 24 16:30:31 2024 |
| committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Sep 24 16:30:31 2024 |
| tree | a7a941bd9b3d9d64bcf7cc0b3866caaea2575786 | |
| parent | e51230311664d302a1b58623296c8565f2de7280 [diff] |
Fix deleted builder detection in chromium/targets_config_verifier. crrev.com/c/5847630 updated the recipe to detect builders that were being deleted and not try to verify them. Unfortunately because it was using a path relative to the repo root, the properties file would never show up as existing, so for any builder where the properties file was modified, it would be treated as though the builder was deleted. This updates the check to correctly root the properties file path to the repo. Bug: 40258588 Change-Id: I90c5a6590edde0b0365c2fbb4ba4058df6aef955 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/5886953 Commit-Queue: Garrett Beaty <gbeaty@google.com> Auto-Submit: Garrett Beaty <gbeaty@google.com> Commit-Queue: Stephanie Kim <kimstephanie@google.com> Reviewed-by: Stephanie Kim <kimstephanie@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.