diff options
author | Jonathan Nieder <jrn@google.com> | 2016-08-16 12:05:09 -0700 |
---|---|---|
committer | Jonathan Nieder <jrn@google.com> | 2016-08-18 17:06:26 -0700 |
commit | 71e4cea6deeccafbd3f4c16f2faa157a0a62750c (patch) | |
tree | b0e889845e7977ce1d7825d6da551b391e7151ba | |
parent | c4c2b066d122d841258ab3efe9adc99a99dc76ea (diff) | |
download | git-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.py | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -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): |