commit | e83330288c3547931b63dd1d4a1164aa44e9f0ba | [log] [tgz] |
---|---|---|
author | Fumitoshi Ukai <ukai@google.com> | Wed Apr 24 06:57:40 2024 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed Apr 24 06:57:40 2024 |
tree | 0d13e717bb9e97d720f5d80c1f2ae6f6fa23cc81 | |
parent | 3133d1557a01148ce9fcf3bb41423d581e0ef6cd [diff] |
siso: don't remove generated file not in out dir. Due to gn restriction, ios build uses a trick of noop step to expose files in build graph. These files are placed out of workspace, but available via symlink in out dir created by `gn gen` time. noop step has outputs, so siso record cmdhash to the output files, but siso hashfs resolves intermediate symlink dir, so cmdhash is recorded in actual real pathname. For cleandead, it checks generated files (which have cmdhash in hashfs state), but such generated files are not matched with files in build graph, so considered as dead file. In the first place, we should not remove files not in out dir by cleandead. Bug: b/336667052 Change-Id: I989bf70d373f3a29c904cf098666da899366734d Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/5481370 Commit-Queue: Fumitoshi Ukai <ukai@google.com> Auto-Submit: Fumitoshi Ukai <ukai@google.com> Reviewed-by: Junji Watanabe <jwata@google.com> Cr-Commit-Position: refs/heads/main@{#64754}
Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, run your tests with test.py:
*_test.py
files to this directory.Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have __init__.py
files in each directory of your module so Python recognizes it as a package.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style, except functions use lowercase_with_underscores
. Use yapf (git cl format
) to autoformat new code.