summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJonathan Nieder <jrn@google.com>2016-08-16 12:05:09 -0700
committerJonathan Nieder <jrn@google.com>2016-08-18 17:06:26 -0700
commit71e4cea6deeccafbd3f4c16f2faa157a0a62750c (patch)
treeb0e889845e7977ce1d7825d6da551b391e7151ba
parentc4c2b066d122d841258ab3efe9adc99a99dc76ea (diff)
downloadgit-repo-71e4cea6deeccafbd3f4c16f2faa157a0a62750c.tar.gz
RepoHook: do not list options twice during hash based approval
Instead of Do you want to allow this script to run (yes/yes-never-ask-again/NO)? (yes/always/NO)? ask Do you want to allow this script to run (yes/always/NO)? Change-Id: I5f5a2d0e88086a8d85e54fb8623a62d74a20956a Signed-off-by: Jonathan Nieder <jrn@google.com>
-rw-r--r--project.py6
1 files changed, 2 insertions, 4 deletions
diff --git a/project.py b/project.py
index 019f29d4..46e06bf8 100644
--- a/project.py
+++ b/project.py
@@ -522,13 +522,11 @@ class RepoHook(object):
522 prompt = ('Repo %s run the script:\n' 522 prompt = ('Repo %s run the script:\n'
523 ' %s\n' 523 ' %s\n'
524 '\n' 524 '\n'
525 'Do you want to allow this script to run ' 525 'Do you want to allow this script to run')
526 '(yes/yes-never-ask-again/NO)? ') % (self._GetMustVerb(),
527 self._script_fullpath)
528 return self._CheckForHookApprovalHelper( 526 return self._CheckForHookApprovalHelper(
529 'approvedhash', 527 'approvedhash',
530 self._GetHash(), 528 self._GetHash(),
531 prompt, 529 prompt % (self._GetMustVerb(), self._script_fullpath),
532 'Scripts have changed since %s was allowed.' % (self._hook_type,)) 530 'Scripts have changed since %s was allowed.' % (self._hook_type,))
533 531
534 def _ExecuteHook(self, **kwargs): 532 def _ExecuteHook(self, **kwargs):