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>
diff --git a/subcmds/sync.py b/subcmds/sync.py
index b752cfb..a66dd6c 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -436,9 +436,7 @@
_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()
