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