This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[PATCH, 1/5] check_GNU_style.sh: Don't use filename prefix for one patch
- From: Tom de Vries <Tom_deVries at mentor dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 12 May 2015 09:27:25 +0200
- Subject: [PATCH, 1/5] check_GNU_style.sh: Don't use filename prefix for one patch
- Authentication-results: sourceware.org; auth=none
Hi,
this patch makes sure that we don't print the patch filename in front of the
error messages, if we're checking just one file. Making the lines shorter makes
things easier to read. And since we're only processing one file, there's no
information lost.
OK for trunk?
Thanks,
- Tom
[PATCH 1/5] check_GNU_style.sh: Don't use filename prefix for one
patch
2015-05-11 Tom de Vries <tom@codesourcery.com>
* check_GNU_style.sh: Don't use a filename prefix if we're only
processing one patch file.
---
contrib/check_GNU_style.sh | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/contrib/check_GNU_style.sh b/contrib/check_GNU_style.sh
index e8d5dca..eeff48f 100755
--- a/contrib/check_GNU_style.sh
+++ b/contrib/check_GNU_style.sh
@@ -36,6 +36,8 @@ EOF
}
test $# -eq 0 && usage
+nfiles=$#
+files="$*"
inp=check_GNU_style.inp
tmp=check_GNU_style.tmp
@@ -44,9 +46,15 @@ tmp=check_GNU_style.tmp
trap "rm -f $inp $tmp" 0
trap "rm -f $inp $tmp ; exit 1" 1 2 3 5 9 13 15
-grep -nH '^+' $* \
- | grep -v ':+++' \
- > $inp
+if [ $nfiles -eq 1 ]; then
+ # There's no need for the file prefix if we're dealing only with one file.
+ format="-n"
+else
+ format="-nH"
+fi
+grep $format '^+' $files \
+ | grep -v ':+++' \
+ > $inp
# Grep
g (){
--
1.9.1