You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2015/10/16 20:40:55 UTC

mesos git commit: Updated pre-commit hook to exit on style or split errors.

Repository: mesos
Updated Branches:
  refs/heads/master 2332fd5b7 -> 887422627


Updated pre-commit hook to exit on style or split errors.

Review: https://reviews.apache.org/r/39404


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/88742262
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/88742262
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/88742262

Branch: refs/heads/master
Commit: 8874226272c2636b02910c62b034481f3664c97f
Parents: 2332fd5
Author: Vinod Kone <vi...@gmail.com>
Authored: Fri Oct 16 11:21:20 2015 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Fri Oct 16 11:34:06 2015 -0700

----------------------------------------------------------------------
 support/hooks/pre-commit | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/88742262/support/hooks/pre-commit
----------------------------------------------------------------------
diff --git a/support/hooks/pre-commit b/support/hooks/pre-commit
index 09ce578..ca9e981 100755
--- a/support/hooks/pre-commit
+++ b/support/hooks/pre-commit
@@ -24,7 +24,7 @@ exec 1>&2
 git diff-index --check --cached $against -- || exit 1
 
 # Check Mesos style.
-exec git diff --cached --name-only --diff-filter=AM | xargs ./support/mesos-style.py
+git diff --cached --name-only --diff-filter=AM | xargs ./support/mesos-style.py || exit 1
 
 # Check that the commits are properly split between mesos, libprocess and stout.
-exec git diff --cached --name-only --diff-filter=AMD | xargs ./support/mesos_split.py
+git diff --cached --name-only --diff-filter=AMD | xargs ./support/mesos_split.py || exit 1