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): |
