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>
		
			
				
	
	
		
			36 lines
		
	
	
		
			1.2 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
			
		
		
	
	
			36 lines
		
	
	
		
			1.2 KiB
		
	
	
	
		
			Bash
		
	
	
		
			Executable File
		
	
	
	
	
#!/bin/sh
 | 
						|
#
 | 
						|
# SPDX-License-Identifier: GPL-2.0-only
 | 
						|
 | 
						|
# DESCR: Check for non-ASCII and unprintable characters
 | 
						|
 | 
						|
LC_ALL=C export LC_ALL
 | 
						|
 | 
						|
INCLUDED_FILES='\.[chsS]$\|\.asl$\|\.cb$\|\.inc$\|Kconfig\|\.ld$|\.txt\|\.hex'
 | 
						|
EXCLUDED_DIRS='^payloads/external/\|^src/vendorcode/\|^Documentation/'
 | 
						|
EXCLUDED_FILES='to-wiki/towiki\.sh$\|vga/vga_font\|video/font\|PDCurses.*x11'
 | 
						|
EXCLUDED_PHRASES='Copyright\|Ported to\|Intel®\|°C\|°F\|Athlon™\|Copyright.*©'
 | 
						|
 | 
						|
# Exit if git isn't present or the code isn't in a git repo
 | 
						|
if [ -z "$(command -v git)" ] || \
 | 
						|
	[ "$(git rev-parse --is-inside-work-tree 2>/dev/null)" != "true" ]
 | 
						|
then
 | 
						|
	exit
 | 
						|
fi
 | 
						|
 | 
						|
# 1. Get the list of files to parse and send them through grep
 | 
						|
# 2. Find any characters that aren't TAB, or space (0x20) to ~ (0x7F)
 | 
						|
#    LF (0x10) isn't included, as it ends the grep line
 | 
						|
# 3. Remove common phrases and names that have been found
 | 
						|
# 4. Run the result through grep again to highlight the issues that were
 | 
						|
#    found.  Without this step, the characters can be difficult to see.
 | 
						|
# shellcheck disable=SC2046
 | 
						|
git grep -lP "[^\t-~]" | \
 | 
						|
	grep "$INCLUDED_FILES" | \
 | 
						|
	grep -v "$EXCLUDED_DIRS" | \
 | 
						|
	grep -v "$EXCLUDED_FILES" | \
 | 
						|
	xargs -I % \
 | 
						|
		grep -n "[^	 -~]" % | \
 | 
						|
		grep -iv "$EXCLUDED_PHRASES" | \
 | 
						|
		grep --color='auto' "[^	 -~]"
 |