96a193afa6
This excludes Devicetree blob files from the list of files to check for
superfluous whitespaces. A DTB file has recently been added in commit
33079b8174 ("lib/device_tree: Add some FDT helper functions").
Change-Id: Ic25ee5361163446370c530cccefa3bf085895d15
Signed-off-by: Maximilian Brune <maximilian.brune@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/82638
Reviewed-by: Eric Lai <ericllai@google.com>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Reviewed-by: Shuo Liu <shuo.liu@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
32 lines
1.1 KiB
Bash
Executable File
32 lines
1.1 KiB
Bash
Executable File
#!/usr/bin/env sh
|
|
#
|
|
# SPDX-License-Identifier: GPL-2.0-only
|
|
|
|
# DESCR: Check for superfluous whitespace in the tree
|
|
|
|
LINTDIR="$(
|
|
cd -- "$(dirname "$0")" > /dev/null 2>&1 || return
|
|
pwd -P
|
|
)"
|
|
# shellcheck source=helper_functions.sh
|
|
. "${LINTDIR}/helper_functions.sh"
|
|
|
|
EXCLUDELIST='^src/vendorcode/|^util/kconfig/|^util/nvidia/cbootimage|^util/goswid|__pycache__|COPYING|LICENSE|README|_shipped$|\.patch$|\.bin$|\.hex$|\.jpg$|\.gif$|\.ttf$|\.woff$|\.png$|\.eot$|\.vbt$|\.ico$|\.md$|\.apcb$|\.dtb$'
|
|
INCLUDELIST="src util payloads Makefile* toolchain.mk tests"
|
|
|
|
# shellcheck disable=SC2086,SC2046
|
|
if uname | grep -qi "linux"; then
|
|
grep -n -H "[[:space:]][[:space:]]*$" \
|
|
$(${FIND_FILES} ${INCLUDELIST} ${FINDOPTS} | \
|
|
grep -Ev "($EXCLUDELIST)" ) | \
|
|
sed -e "s,^.*$,File & has lines ending with whitespace.,"
|
|
else
|
|
# The above form is much (100x) faster, but doesn't work
|
|
# on all systems. A for loop also works but takes 30% longer
|
|
${FIND_FILES} ${INCLUDELIST} ${FINDOPTS}| \
|
|
grep -Ev "($EXCLUDELIST)" | \
|
|
xargs -I % \
|
|
grep -l "[[:space:]][[:space:]]*$" % | \
|
|
sed -e "s,^.*$,File & has lines ending with whitespace.,"
|
|
fi
|