diff options
| author | Mike Frysinger <vapier@google.com> | 2020-02-17 14:36:08 -0500 |
|---|---|---|
| committer | David Pursehouse <dpursehouse@collab.net> | 2020-02-19 00:24:43 +0000 |
| commit | c58ec4dba102d88fec67e833eb8421202eb4c1ea (patch) | |
| tree | c8c84e03f9359ae9dd126854069d34207c298831 /subcmds/selfupdate.py | |
| parent | e1191b3adb22e0b406db87691b13ddab2c236267 (diff) | |
| download | git-repo-c58ec4dba102d88fec67e833eb8421202eb4c1ea.tar.gz | |
avoid negative variables
Trying to use booleans with names like "no_xxx" are hard to follow due
to the double negatives. Invert all of them so we only have positive
meanings to follow.
Change-Id: Ifd37d0368f97034d94aa2cf38db52c723ac0c6ed
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/255493
Tested-by: Mike Frysinger <vapier@google.com>
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Diffstat (limited to 'subcmds/selfupdate.py')
| -rw-r--r-- | subcmds/selfupdate.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/selfupdate.py b/subcmds/selfupdate.py index 4817a862..bf6256ab 100644 --- a/subcmds/selfupdate.py +++ b/subcmds/selfupdate.py | |||
| @@ -40,7 +40,7 @@ need to be performed by an end-user. | |||
| 40 | def _Options(self, p): | 40 | def _Options(self, p): |
| 41 | g = p.add_option_group('repo Version options') | 41 | g = p.add_option_group('repo Version options') |
| 42 | g.add_option('--no-repo-verify', | 42 | g.add_option('--no-repo-verify', |
| 43 | dest='no_repo_verify', action='store_true', | 43 | dest='repo_verify', default=True, action='store_false', |
| 44 | help='do not verify repo source code') | 44 | help='do not verify repo source code') |
| 45 | g.add_option('--repo-upgraded', | 45 | g.add_option('--repo-upgraded', |
| 46 | dest='repo_upgraded', action='store_true', | 46 | dest='repo_upgraded', action='store_true', |
| @@ -60,5 +60,5 @@ need to be performed by an end-user. | |||
| 60 | 60 | ||
| 61 | rp.bare_git.gc('--auto') | 61 | rp.bare_git.gc('--auto') |
| 62 | _PostRepoFetch(rp, | 62 | _PostRepoFetch(rp, |
| 63 | no_repo_verify=opt.no_repo_verify, | 63 | repo_verify=opt.repo_verify, |
| 64 | verbose=True) | 64 | verbose=True) |
