Skip to content

Commit

Permalink
checkpatch: Ignore android/abi* files during analysis
Browse files Browse the repository at this point in the history
The abi related files are a regular source for false positives. Hence
ignore them altogether.

Signed-off-by: Matthias Maennich <[email protected]>
Change-Id: Iaddaa4237819fb69456043e5767889c972f1a983
  • Loading branch information
metti committed Feb 22, 2021
1 parent 031de8c commit e9f1b5a
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion static_analysis/checkpatch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -94,9 +94,15 @@ echo "Using --git_sha1: ${GIT_SHA1}"

# Generate patch file from git.
cd ${KERNEL_DIR}
git format-patch --quiet -o "${PATCH_DIR}" "${GIT_SHA1}^1..${GIT_SHA1}"
git format-patch --quiet -o "${PATCH_DIR}" "${GIT_SHA1}^1..${GIT_SHA1}" -- \
':!android/abi*'
PATCH_FILE="${PATCH_DIR}/*.patch"

if ! `stat -t ${PATCH_FILE} >/dev/null 2>&1`; then
echo "Patch empty (probably due to suppressions). Skipping analysis."
exit 0
fi

# 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
Expand Down

0 comments on commit e9f1b5a

Please sign in to comment.