From 2273f46cb36893d20ab77d4dde1461d670d88e2c Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Fri, 5 Nov 2021 15:10:33 -0400 Subject: [PATCH] sync: fix --tags option This has been broken since it was added where --tags was actually the same as --no-tags. Oddly, it was copied from init where the logic is correct. Bug: https://crbug.com/gerrit/12401 Change-Id: I15b89da1a655176a11bebc22573b25c728055328 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/322955 Reviewed-by: Andrew Lamb Tested-by: Mike Frysinger --- subcmds/sync.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/subcmds/sync.py b/subcmds/sync.py index c2303232..d4e302ac 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -238,12 +238,11 @@ later is required to fix a server side protocol bug. p.add_option('--no-use-superproject', action='store_false', dest='use_superproject', help='disable use of manifest superprojects') - p.add_option('--tags', - action='store_false', + p.add_option('--tags', action='store_true', help='fetch tags') p.add_option('--no-tags', dest='tags', action='store_false', - help="don't fetch tags") + help="don't fetch tags (default)") p.add_option('--optimized-fetch', dest='optimized_fetch', action='store_true', help='only fetch projects fixed to sha1 if revision does not exist locally')