commit 3d27c71dd928bd345257055cdf2a6be8f8a09bed
author Mike Frysinger <vapier@google.com> Fri Aug 19 06:48:23 2022 -0400
committer Mike Frysinger <vapier@google.com> Mon Aug 22 21:01:35 2022 +0000
tree fdfdc4dd7760f6923c2467a49b012c7ea1974f3f
parent 488d54d4eea1e0bae5c6056ae628b783d4f716c0

init: hide identify spam when reinitializing

We don't want to keep showing the user config notice when reinitializing
existing checkouts, so hide it.

Change-Id: Id40610bd683396cbff7e1aefc092c8b77c397783
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/343536
Tested-by: Mike Frysinger <vapier@google.com>
Reviewed-by: LaMont Jones <lamontjones@google.com>

subcmds/init.py

diff --git a/subcmds/init.py b/subcmds/init.py
index 803f39a..0c979cd 100644
--- a/subcmds/init.py
+++ b/subcmds/init.py
@@ -148,7 +148,7 @@
       return value
     return a
 
-  def _ShouldConfigureUser(self, opt):
+  def _ShouldConfigureUser(self, opt, existing_checkout):
     gc = self.client.globalConfig
     mp = self.manifest.manifestProject
 
@@ -160,7 +160,7 @@
       mp.config.SetString('user.name', gc.GetString('user.name'))
       mp.config.SetString('user.email', gc.GetString('user.email'))
 
-    if not opt.quiet:
+    if not opt.quiet and not existing_checkout or opt.verbose:
       print()
       print('Your identity is: %s <%s>' % (mp.config.GetString('user.name'),
                                            mp.config.GetString('user.email')))
@@ -325,7 +325,7 @@
     self._SyncManifest(opt)
 
     if os.isatty(0) and os.isatty(1) and not self.manifest.IsMirror:
-      if opt.config_name or self._ShouldConfigureUser(opt):
+      if opt.config_name or self._ShouldConfigureUser(opt, existing_checkout):
         self._ConfigureUser(opt)
       self._ConfigureColor()