commit 2273f46cb36893d20ab77d4dde1461d670d88e2c
author Mike Frysinger <vapier@google.com> Fri Nov 05 15:10:33 2021 -0400
committer Mike Frysinger <vapier@google.com> Fri Nov 05 20:13:02 2021 +0000
tree 2e7dd4cc70754b179b4280629e32132878e8ca8d
parent 7b9b251a5e86eed8d346ffe84dc22b8f1cb1a05b

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 <andrewlamb@google.com>
Tested-by: Mike Frysinger <vapier@google.com>

subcmds/sync.py

diff --git a/subcmds/sync.py b/subcmds/sync.py
index c230323..d4e302a 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -238,12 +238,11 @@
     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')