diff options
author | Erik Staats <estaats@google.com> | 2024-04-17 21:03:10 +0000 |
---|---|---|
committer | Matthias Männich <maennich@google.com> | 2024-04-17 23:32:23 +0000 |
commit | 05bdd1b3fae49adda5bd2a459ba7ef905d524f6d (patch) | |
tree | 5f6122b9f52ec37900196df868c1554a5109cedc | |
parent | 78dabc5185a1e3d9234d8cbcea2ae75f9bea9aab (diff) | |
download | build-05bdd1b3fae49adda5bd2a459ba7ef905d524f6d.tar.gz |
kleaf: checkpatch_presubmit: Use --no-tree option
Running with --no-tree prevents spdxcheck.py from being run and
generating an error.
Bug: 199237323
Bug: 327484174
Test: Verified no errors are encountered when running
checkpatch_presubmit on a patch with a SPDX header.
Change-Id: I7e19f400598650f260a2721f76106bd7d31e8262
Signed-off-by: Erik Staats <estaats@google.com>
-rwxr-xr-x | kleaf/impl/checkpatch.sh | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/kleaf/impl/checkpatch.sh b/kleaf/impl/checkpatch.sh index a9dedfd..068bd1d 100755 --- a/kleaf/impl/checkpatch.sh +++ b/kleaf/impl/checkpatch.sh @@ -178,6 +178,10 @@ echo "========================================================" >> "${MY_RESULTS echo "${DIR}: ${GIT_SHA1}" >> "${MY_RESULTS_PATH}" echo "========================================================" >> "${MY_RESULTS_PATH}" +# TODO: b/199237323 - Run with --no-tree option for now to avoid spdxcheck.py +# failures. A better fix would be to remove spdxcheck.py. +CHECKPATCH_ARGS+=(--no-tree) + # Delay exit on non-zero checkpatch.pl return code so we can finish logging. # Note, it's tricky to ignore this exit code completely and instead return only |