From 511a0e54f5801a3f36c00fac478a596d83867d10 Mon Sep 17 00:00:00 2001
From: Mike Frysinger <vapier@google.com>
Date: Fri, 12 Mar 2021 20:03:51 -0500
Subject: [PATCH] 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 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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 @@ later is required to fix a server side protocol bug.
 
     pm.end()
 
-    return ret
+    return ret and not err_results
 
   def _GCProjects(self, projects, opt, err_event):
     gc_gitdirs = {}