diff options
author | Stefan Müller-Klieser <s.mueller-klieser@phytec.de> | 2019-08-30 10:20:15 +0200 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2019-08-30 16:29:01 +0000 |
commit | 46702eddc7b5f38391e121e47f0ab4dbd6fbdf58 (patch) | |
tree | 6f73c5a549b7ea1d7f00255542b610e3e0ea0701 /subcmds/sync.py | |
parent | ae6cb08ae52d488a4cc6892f811c1c1acf8c3c12 (diff) | |
download | git-repo-46702eddc7b5f38391e121e47f0ab4dbd6fbdf58.tar.gz |
sync: fix deprecated command line option -fv1.13.5.1
In commit d9e5cf0e ("sync: invert --force-broken with --fail-fast") the
force-broken option has been deprecated. Accidentally the option has
been changed from Boolean to Value. This breaks all users of repo with:
main.py: error: -f option requires an argument
This is easy to avoid by keeping the type.
Signed-off-by: Stefan Müller-Klieser <s.mueller-klieser@phytec.de>
Change-Id: Ia8b589cf41ac756d10c61e17ec8d76ba8f7031f9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/235043
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Reviewed-by: Mike Frysinger <vapier@google.com>
Tested-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'subcmds/sync.py')
-rw-r--r-- | subcmds/sync.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 5655a1e6..f1fe88d8 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
@@ -199,6 +199,7 @@ later is required to fix a server side protocol bug. | |||
199 | self.jobs = 1 | 199 | self.jobs = 1 |
200 | 200 | ||
201 | p.add_option('-f', '--force-broken', | 201 | p.add_option('-f', '--force-broken', |
202 | dest='force_broken', action='store_true', | ||
202 | help='obsolete option (to be deleted in the future)') | 203 | help='obsolete option (to be deleted in the future)') |
203 | p.add_option('--fail-fast', | 204 | p.add_option('--fail-fast', |
204 | dest='fail_fast', action='store_true', | 205 | dest='fail_fast', action='store_true', |