diff options
| author | Joas Schilling <nickvergessen@gmx.de> | 2014-03-01 19:16:58 +0100 |
|---|---|---|
| committer | Joas Schilling <nickvergessen@gmx.de> | 2014-03-01 19:16:58 +0100 |
| commit | d636cb0fc9fc37fe3c806702c969c7b694ae439f (patch) | |
| tree | 78ea3b42c89740f5a43fd2aea1a52f2690111918 /git-tools/commit-msg-hook-range.sh | |
| parent | e9ed9cd942042616d598bebdbfd91cfd40bedacf (diff) | |
| parent | 9f4908f0f2896d60c348f4e52ef4a200a8404bd7 (diff) | |
| download | forums-d636cb0fc9fc37fe3c806702c969c7b694ae439f.tar forums-d636cb0fc9fc37fe3c806702c969c7b694ae439f.tar.gz forums-d636cb0fc9fc37fe3c806702c969c7b694ae439f.tar.bz2 forums-d636cb0fc9fc37fe3c806702c969c7b694ae439f.tar.xz forums-d636cb0fc9fc37fe3c806702c969c7b694ae439f.zip | |
Merge branch 'develop-olympus' into develop
* develop-olympus:
[ticket/11509] Exit with the expected (non-zero) exit status on failure.
[ticket/11509] Output which commit is being inspected.
[ticket/11509] Three dots means sym. difference, but we only want new commits.
Conflicts:
.travis.yml
Diffstat (limited to 'git-tools/commit-msg-hook-range.sh')
| -rwxr-xr-x | git-tools/commit-msg-hook-range.sh | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/git-tools/commit-msg-hook-range.sh b/git-tools/commit-msg-hook-range.sh index 66628c1d17..2b408c3e79 100755 --- a/git-tools/commit-msg-hook-range.sh +++ b/git-tools/commit-msg-hook-range.sh @@ -5,26 +5,47 @@ # # Calls the git commit-msg hook on all non-merge commits in a given commit range. # -set -e if [ "$#" -ne 1 ]; then - echo "Expected one argument (commit range, e.g. eef1b586...1666476b)." + echo "Expected one argument (commit range, e.g. phpbb/develop..ticket/12345)." exit fi DIR=$(dirname "$0") -COMMIT_MSG_HOOK_PATH="$DIR/hooks/commit-msg" - COMMIT_RANGE="$1" +COMMIT_MSG_HOOK_PATH="$DIR/hooks/commit-msg" +COMMIT_MSG_HOOK_FATAL=$(git config --bool phpbb.hooks.commit-msg.fatal 2> /dev/null) +git config phpbb.hooks.commit-msg.fatal true +EXIT_STATUS=0 for COMMIT_HASH in $(git rev-list --no-merges "$COMMIT_RANGE") do + echo "Inspecting commit message of commit $COMMIT_HASH" + # The git commit-msg hook takes a path to a file containing a commit # message. So we have to extract the commit message into a file first, # which then also needs to be deleted after our work is done. COMMIT_MESSAGE_PATH="$DIR/commit_msg.$COMMIT_HASH" git log -n 1 --pretty=format:%B "$COMMIT_HASH" > "$COMMIT_MESSAGE_PATH" + + # Invoke hook on commit message file. "$COMMIT_MSG_HOOK_PATH" "$COMMIT_MESSAGE_PATH" + + # If any commit message hook complains with a non-zero exit status, we + # will send a non-zero exit status upstream. + if [ $? -ne 0 ] + then + EXIT_STATUS=1 + fi + rm "$COMMIT_MESSAGE_PATH" done + +# Restore phpbb.hooks.commit-msg.fatal config +if [ -n "$COMMIT_MSG_HOOK_FATAL" ] +then + git config phpbb.hooks.commit-msg.fatal "$COMMIT_MSG_HOOK_FATAL" +fi + +exit $EXIT_STATUS |