sync: fix handling of -f and local checkouts

The partial clone rework (commit 745be2ede1e67421275afc00c04d996d9d6908ee
"Add support for partial clone") changed the behavior when a single repo
hit a failure: it would always call sys.exit() immediately.  This isn't
even necessary as we already pass down an error event object which the
workers set and the parent checks.  Just delete the exit entirely.

Change-Id: Id72d8642aefa2bde24e1a438dbe102c3e3cabf48
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/233552
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Tested-by: Mike Frysinger <vapier@google.com>

1 files changed, 1 insertions, 3 deletions

diff --git a/subcmds/sync.py b/subcmds/sync.py
index b752cfbe..a66dd6cd 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -436,9 +436,7 @@ later is required to fix a server side protocol bug.
           _CheckoutOne docstring for details.
     """
     try:
-      success = self._CheckoutOne(opt, project, *args, **kwargs)
-      if not success:
-        sys.exit(1)
+      return self._CheckoutOne(opt, project, *args, **kwargs)
     finally:
       sem.release()