.apcb files are binary configuration data and not human readable; exclude them from license, newline, and whitespace checks. Change-Id: Idc1ddd5067cb97ef8b5758a0b8bf040d1e421871 Signed-off-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/79589 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martin.roth@amd.corp-partner.google.com>
		
			
				
	
	
		
			68 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			68 lines
		
	
	
		
			1.8 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
#!/usr/bin/env sh
 | 
						|
#
 | 
						|
# SPDX-License-Identifier: GPL-2.0-only
 | 
						|
 | 
						|
# DESCR: Check that files end with a single newline
 | 
						|
 | 
						|
LINTDIR="$(
 | 
						|
  cd -- "$(dirname "$0")" > /dev/null 2>&1 || return
 | 
						|
  pwd -P
 | 
						|
)"
 | 
						|
 | 
						|
# shellcheck source=helper_functions.sh
 | 
						|
. "${LINTDIR}/helper_functions.sh"
 | 
						|
 | 
						|
PIDS=""
 | 
						|
INCLUDED_DIRS_AND_FILES='util/* src/* payloads/* configs/* Makefile *.inc'
 | 
						|
EXCLUDED_DIRS='src/vendorcode/\|cbfstool/lzma/\|cbfstool/lz4/\|Documentation/\|build/\|3rdparty/\|\.git/\|coreboot-builds/\|util/nvidia/cbootimage/\|^util/goswid/vendor'
 | 
						|
EXCLUDED_FILES='\.gif$\|\.jpg$\|\.cksum$\|\.bin$\|\.vbt$\|\.hex$\|\.ico$\|\.o$\|\.bz2$\|\.xz$\|^.tmpconfig\|\.pyc$\|_shipped$\|sha256$\|\.png$\|\.patch$\|\.apcb$'
 | 
						|
 | 
						|
HAVE_FILE=$(command -v file 1>/dev/null 2>&1; echo $?)
 | 
						|
 | 
						|
is_eligible_executable() {
 | 
						|
	if [ "$HAVE_FILE" -ne 0 ]; then
 | 
						|
		return 1
 | 
						|
	fi
 | 
						|
	if { LC_ALL=C; file --brief "$filename" | grep -Eqw \
 | 
						|
		"^(Bourne shell|POSIX shell|Perl|Python) script"; };
 | 
						|
	then
 | 
						|
		return 0
 | 
						|
	else
 | 
						|
		return 1
 | 
						|
	fi
 | 
						|
}
 | 
						|
 | 
						|
test_for_final_newline() {
 | 
						|
	while read filename; do
 | 
						|
		# Only check regular files and script executables
 | 
						|
		if [ -f "$filename" ] && { [ ! -x "$filename" ] || \
 | 
						|
			is_eligible_executable "$filename"; };
 | 
						|
		then
 | 
						|
			# Verify that there is a newline at the end
 | 
						|
			# $() strips trailing newlines
 | 
						|
			if [ -n "$(tail -c 1 "$filename")" ]; then
 | 
						|
				echo "$filename has no final newline."
 | 
						|
 | 
						|
			# Verify that the file ends with only a single newline
 | 
						|
			# and that the file isn't empty
 | 
						|
			elif [ -z "$(tail -c 2 "$filename")" ] && \
 | 
						|
			     [ -n "$(head -n 5 "$filename")" ]; then
 | 
						|
				echo "$filename has multiple final newlines."
 | 
						|
			fi
 | 
						|
		fi
 | 
						|
	done
 | 
						|
}
 | 
						|
 | 
						|
for directory in $INCLUDED_DIRS_AND_FILES ; do
 | 
						|
	${FIND_FILES} "${directory}" | sed 's|^\./||' | sort | \
 | 
						|
		grep -v "$EXCLUDED_DIRS" | \
 | 
						|
		grep -v "$EXCLUDED_FILES" | \
 | 
						|
		test_for_final_newline &
 | 
						|
	PIDS="$PIDS $!"
 | 
						|
done
 | 
						|
 | 
						|
# wait for tests to finish.
 | 
						|
for pid in $PIDS; do
 | 
						|
	wait "$pid"
 | 
						|
done
 |