Stefan thinks they don't add value. Command used: sed -i -e '/file is part of /d' $(git grep "file is part of " |egrep ":( */\*.*\*/\$|#|;#|-- | *\* )" | cut -d: -f1 |grep -v crossgcc |grep -v gcov | grep -v /elf.h |grep -v nvramtool) The exceptions are for: - crossgcc (patch file) - gcov (imported from gcc) - elf.h (imported from GNU's libc) - nvramtool (more complicated header) The removed lines are: - fmt.Fprintln(f, "/* This file is part of the coreboot project. */") -# This file is part of a set of unofficial pre-commit hooks available -/* This file is part of coreboot */ -# This file is part of msrtool. -/* This file is part of msrtool. */ - * This file is part of ncurses, designed to be appended after curses.h.in -/* This file is part of pgtblgen. */ - * This file is part of the coreboot project. - /* This file is part of the coreboot project. */ -# This file is part of the coreboot project. -# This file is part of the coreboot project. -## This file is part of the coreboot project. --- This file is part of the coreboot project. -/* This file is part of the coreboot project */ -/* This file is part of the coreboot project. */ -;## This file is part of the coreboot project. -# This file is part of the coreboot project. It originated in the - * This file is part of the coreinfo project. -## This file is part of the coreinfo project. - * This file is part of the depthcharge project. -/* This file is part of the depthcharge project. */ -/* This file is part of the ectool project. */ - * This file is part of the GNU C Library. - * This file is part of the libpayload project. -## This file is part of the libpayload project. -/* This file is part of the Linux kernel. */ -## This file is part of the superiotool project. -/* This file is part of the superiotool project */ -/* This file is part of uio_usbdebug */ Change-Id: I82d872b3b337388c93d5f5bf704e9ee9e53ab3a9 Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/41194 Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
		
			
				
	
	
		
			46 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			46 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
#!/bin/sh
 | 
						|
#
 | 
						|
# SPDX-License-Identifier: GPL-2.0-only
 | 
						|
 | 
						|
# DESCR: Checkpatch on .c, .h, & Kconfig files in the tree
 | 
						|
 | 
						|
LC_ALL=C export LC_ALL
 | 
						|
 | 
						|
# GNU BRE syntax list of files to examine
 | 
						|
INCLUDED_FILES='.*\.[ch]\|Kconfig.*$'
 | 
						|
 | 
						|
EXCLUDED_DIRS="^payloads/libpayload/util/kconfig\|\
 | 
						|
^payloads/libpayload/curses/PDCurses\|\
 | 
						|
^util/crossgcc/patches\|\
 | 
						|
^util/inteltool\|\
 | 
						|
^util/kconfig\|\
 | 
						|
^util/superiotool\|\
 | 
						|
^src/vendorcode\|\
 | 
						|
^Documentation"
 | 
						|
 | 
						|
opts="--max-line-length 96"
 | 
						|
 | 
						|
# default: test src and util
 | 
						|
if [ "$1" = "" ]; then
 | 
						|
	INCLUDED_DIRS="src util"
 | 
						|
# special mode: take diff from stdin, but exclude the dirs
 | 
						|
elif [ "$1" = "diff" ]; then
 | 
						|
	args=$( echo $EXCLUDED_DIRS | \
 | 
						|
		sed -e 's,\\|, ,g' -e 's,\^,--exclude=,g' )
 | 
						|
	util/lint/checkpatch.pl --quiet --no-signoff $opts $args -
 | 
						|
	exit $?
 | 
						|
# Space separated list of directories to test
 | 
						|
else
 | 
						|
	INCLUDED_DIRS="$1"
 | 
						|
fi
 | 
						|
 | 
						|
# We want word splitting here, so disable the shellcheck warnings
 | 
						|
# shellcheck disable=SC2046,SC2086
 | 
						|
FILELIST=$( git ls-files $INCLUDED_DIRS | \
 | 
						|
		grep $INCLUDED_FILES | \
 | 
						|
		grep -v $EXCLUDED_DIRS )
 | 
						|
 | 
						|
for FILE in $FILELIST; do
 | 
						|
	util/lint/checkpatch.pl --show-types --file --quiet $opts "$FILE"
 | 
						|
done
 |