diff options
author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2012-05-14 21:05:29 -0400 |
---|---|---|
committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2012-05-14 21:05:47 -0400 |
commit | 8d043e6e1febfa0f49d526f40727beda8a262a6e (patch) | |
tree | ddbc07e87c0645172d47dd1925ba9ff66f10bae6 | |
parent | 34d557143054a72e7ab32ceeb97ccc9bd6422fc5 (diff) | |
download | longterm-queue-2.6.34-8d043e6e1febfa0f49d526f40727beda8a262a6e.tar.gz |
make reviewbot spit out any reason why a patch fails sanity check
It searches for my initials, followed by ":" -- what I'll put above
my SOB line. Limited to two lines of text, which covers 99% of 'em.
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
-rwxr-xr-x | scripts/reviewbot | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/scripts/reviewbot b/scripts/reviewbot index 40ba029..89e8b54 100755 --- a/scripts/reviewbot +++ b/scripts/reviewbot @@ -10,6 +10,10 @@ PATCH=$1 +# If a patch appears "different", then look for this in the commit log +# to see if the gomer messing with it explained why. +GOMER=PG: + if [ -z "$1" ]; then echo must supply patchname exit 1 @@ -126,6 +130,13 @@ if [ $SOB_PCOUNT -ne $SOB_CCOUNT ];then fi RETVAL=1 fi - + + +# Look for explanation from mangler as to why it might fail sanity check. +if [ $RETVAL != 0 ]; then + if [ -z "$QUIET" ]; then + grep -A1 $GOMER $1 + fi +fi exit $RETVAL |