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' "[^	 -~]"
 |