You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by be...@apache.org on 2013/07/12 04:26:18 UTC

[2/2] git commit: Check for uncommitted changes before running post-reviews.

Check for uncommitted changes before running post-reviews.

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


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

Branch: refs/heads/master
Commit: fec1a180c3ad5c09db9bd1bf5e1151d40c8fab4a
Parents: 9c34dd3
Author: Benjamin Hindman <be...@gmail.com>
Authored: Thu Jul 11 18:56:34 2013 -0700
Committer: Benjamin Hindman <be...@gmail.com>
Committed: Thu Jul 11 19:26:00 2013 -0700

----------------------------------------------------------------------
 support/post-reviews.py | 8 ++++++++
 1 file changed, 8 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/fec1a180/support/post-reviews.py
----------------------------------------------------------------------
diff --git a/support/post-reviews.py b/support/post-reviews.py
index 6d886ca..89407e9 100755
--- a/support/post-reviews.py
+++ b/support/post-reviews.py
@@ -55,6 +55,14 @@ def execute(command, ignore_errors=False):
 
 
 # TODO(benh): Make sure this is a git repository, apologize if not.
+
+# Don't do anything if people have uncommitted changes.
+diff_stat = execute(['git', 'diff', '--shortstat']).strip()
+
+if diff_stat:
+  print 'Please commit or stash any changes before using post-reviews!'
+  sys.exit(1)
+
 top_level_dir = execute(['git', 'rev-parse', '--show-toplevel']).strip()
 
 repository = 'git://git.apache.org/incubator-mesos.git'