[tor-commits] [tor/master] pre-push hook: Only run practracker when a special file is present

[asn at torproject.org]

commit 71e5af0221568b36d128be88c958a7de018ebcb3
Author: Nick Mathewson <nickm at torproject.org>
Date:   Thu Aug 8 11:32:11 2019 -0400

    pre-push hook: Only run practracker when a special file is present
    
    Closes ticket 30979.
---
 changes/ticket30979                                    |  5 +++++
 scripts/git/pre-push.git-hook                          | 13 ++++++++-----
 scripts/maint/practracker/.enable_practracker_in_hooks |  1 +
 3 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/changes/ticket30979 b/changes/ticket30979
new file mode 100644
index 000000000..8ae9b3c41
--- /dev/null
+++ b/changes/ticket30979
@@ -0,0 +1,5 @@
+  o Minor features (git hooks):
+    - Our pre-push git hook now checks for a special file
+      before running practracker, so that it only runs on branches
+      that are based on master.
+      Closes ticket 30979.
diff --git a/scripts/git/pre-push.git-hook b/scripts/git/pre-push.git-hook
index 71abc9aa2..40a3bffa7 100755
--- a/scripts/git/pre-push.git-hook
+++ b/scripts/git/pre-push.git-hook
@@ -28,10 +28,14 @@ if [ -x "$workdir/.git/hooks/pre-commit" ]; then
   fi
 fi
 
-if [ -e scripts/maint/practracker/practracker.py ]; then
-  if ! python3 ./scripts/maint/practracker/practracker.py "$workdir"; then
-    exit 1
-  fi
+PT_DIR=scripts/maint/practracker
+
+if [ -e "${PT_DIR}/practracker.py" ]; then
+    if [ -e "${PT_DIR}/.enable_practracker_in_hooks" ]; then
+        if ! python3 "${PT_DIR}/practracker.py" "$workdir"; then
+            exit 1
+        fi
+    fi
 fi
 
 remote="$1"
@@ -104,4 +108,3 @@ do
 done
 
 exit 0
-
diff --git a/scripts/maint/practracker/.enable_practracker_in_hooks b/scripts/maint/practracker/.enable_practracker_in_hooks
new file mode 100644
index 000000000..a9e707f5d
--- /dev/null
+++ b/scripts/maint/practracker/.enable_practracker_in_hooks
@@ -0,0 +1 @@
+This file is present to tell our git hooks to run practracker on this branch.