diff options
author | Mike Frysinger <vapier@google.com> | 2021-03-12 20:03:51 -0500 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2021-03-15 16:54:23 +0000 |
commit | 511a0e54f5801a3f36c00fac478a596d83867d10 (patch) | |
tree | eeee3c2014b563bfdbede9cdb7359cae348a5db3 | |
parent | 8da7b6fc65f5f5eedbcd251bb7c423148101e91b (diff) | |
download | git-repo-511a0e54f5801a3f36c00fac478a596d83867d10.tar.gz |
sync: fix reporting of failed local checkoutsv2.13.6
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>
-rw-r--r-- | subcmds/sync.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 3ad0627d..2953ee3b 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
@@ -528,7 +528,7 @@ later is required to fix a server side protocol bug. | |||
528 | 528 | ||
529 | pm.end() | 529 | pm.end() |
530 | 530 | ||
531 | return ret | 531 | return ret and not err_results |
532 | 532 | ||
533 | def _GCProjects(self, projects, opt, err_event): | 533 | def _GCProjects(self, projects, opt, err_event): |
534 | gc_gitdirs = {} | 534 | gc_gitdirs = {} |