diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2015-04-10 19:12:16 -0700 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2015-04-10 19:12:16 -0700 |
commit | 89b2bf617fa0619d57e8875d6d15aa765d765482 (patch) | |
tree | 39db8c7d7ea7fb1d466d53bd20dbe8b847955543 /build-aux | |
parent | c0984249eb65641a0876594183c80fd8e6b37962 (diff) |
Have commit-msg report commit failure
* build-aux/git-hooks/commit-msg: If the commit is aborted,
say so. Simplify by doing this at the end. Problem reported
by Eli Zaretskii in:
http://lists.gnu.org/archive/html/emacs-devel/2015-04/msg00566.html
Diffstat (limited to 'build-aux')
-rwxr-xr-x | build-aux/git-hooks/commit-msg | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/build-aux/git-hooks/commit-msg b/build-aux/git-hooks/commit-msg index 13a05351de..6721d53a58 100755 --- a/build-aux/git-hooks/commit-msg +++ b/build-aux/git-hooks/commit-msg @@ -63,7 +63,7 @@ exec $awk ' /^#/ { next } !/^.*$/ { - print "Invalid character (not UTF-8) in commit message; see 'CONTRIBUTE'" + print "Invalid character (not UTF-8) in commit message" status = 1 } @@ -77,13 +77,13 @@ exec $awk ' sub(/^squash! /, "") if ($0 ~ "^" space) { - print "White space at start of commit message'\''s first line; see 'CONTRIBUTE'" + print "White space at start of commit message'\''s first line" status = 1 } } nlines == 2 && $0 ~ non_space { - print "Nonempty second line in commit message; see 'CONTRIBUTE'" + print "Nonempty second line in commit message" status = 1 } @@ -97,30 +97,33 @@ exec $awk ' } 78 < length && $0 ~ space { - print "Line longer than 78 characters in commit message; see 'CONTRIBUTE'" + print "Line longer than 78 characters in commit message" status = 1 } 140 < length { - print "Word longer than 140 characters in commit message; see 'CONTRIBUTE'" + print "Word longer than 140 characters in commit message" status = 1 } /^Signed-off-by: / { - print "'\''Signed-off-by:'\'' in commit message; see 'CONTRIBUTE'" + print "'\''Signed-off-by:'\'' in commit message" status = 1 } $0 ~ non_print { - print "Unprintable character in commit message; see 'CONTRIBUTE'" + print "Unprintable character in commit message" status = 1 } END { if (nlines == 0) { - print "Empty commit message; see 'CONTRIBUTE'" + print "Empty commit message" status = 1 } + if (status != 0) { + print "Commit aborted; please see the file 'CONTRIBUTE'" + } exit status } ' <"$1" |