From a34186e4813170f3c71ec51c740cd571c79e12b5 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Wed, 7 Aug 2019 18:07:31 -0400 Subject: 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 Tested-by: Mike Frysinger --- subcmds/sync.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'subcmds/sync.py') 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() -- cgit v1.2.3-54-g00ecf