commit 511a0e54f5801a3f36c00fac478a596d83867d10
author Mike Frysinger <vapier@google.com> Fri Mar 12 20:03:51 2021 -0500
committer Mike Frysinger <vapier@google.com> Mon Mar 15 16:54:23 2021 +0000
tree eeee3c2014b563bfdbede9cdb7359cae348a5db3
parent 8da7b6fc65f5f5eedbcd251bb7c423148101e91b

sync: fix reporting of failed local checkouts

The refactor to multiprocessing broke status reporting slightly when
checking out projects.  Make sure we mark the step as failed if any
of the projects failed, not just when --fail-fast is set.

Change-Id: I0efb56ce83b068b2c334046df3fef23d797599c9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/299882
Reviewed-by: Michael Mortensen <mmortensen@google.com>
Tested-by: Mike Frysinger <vapier@google.com>

subcmds/sync.py

diff --git a/subcmds/sync.py b/subcmds/sync.py
index 3ad0627..2953ee3 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -528,7 +528,7 @@
 
     pm.end()
 
-    return ret
+    return ret and not err_results
 
   def _GCProjects(self, projects, opt, err_event):
     gc_gitdirs = {}