commit 219431e1c9868e8d5e51308d0c42bb084c510ea9
author Mike Frysinger <vapier@google.com> Tue May 04 19:46:09 2021 -0400
committer Mike Frysinger <vapier@google.com> Wed May 05 02:42:23 2021 +0000
tree 7b3713b58e81266039d80a9a4b8d931d88e75276
parent 5ba80d404cd6d5dc6d1ccfcfc5f8fc560d82be06

sync: fix recursive fetching

Commit b2fa30a2b891b22c173c960a67bf38ccbba8de1b ("sync: switch network
fetch to multiprocessing") accidentally changed the variable passed to
the 2nd fetch call from |missing| to |to_fetch| due to a copy & paste
of the earlier changed logic.  Undo that to fix git submodule fetching.

Bug: https://crbug.com/gerrit/14489
Change-Id: I627954f80fd2e80d9d5809b530aa6b0ef9260abb
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/305262
Reviewed-by: Raman Tenneti <rtenneti@google.com>
Tested-by: Mike Frysinger <vapier@google.com>

subcmds/sync.py

diff --git a/subcmds/sync.py b/subcmds/sync.py
index 13ae0fe..d3c326a 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -959,7 +959,7 @@
         if previously_missing_set == missing_set:
           break
         previously_missing_set = missing_set
-        success, new_fetched = self._Fetch(to_fetch, opt, err_event)
+        success, new_fetched = self._Fetch(missing, opt, err_event)
         if not success:
           err_event.set()
         fetched.update(new_fetched)