From 05638bf77193d53a8568600345733265e4c6a42c Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 4 May 2021 15:33:31 -0400 Subject: [PATCH] sync: use manifest_name passed in Commit fb527e3f522a9f6ae05ee1fd707fb0b45c9190f2 ("sync: create dedicated manifest project update func") refactored code from the main body into a dedicated method. The manifest_name was passed as an argument, but never used it, and instead reaches back out to the command line options. This ignores the logic in the main loop where manifest_name might have changed (like when using smart sync). Change-Id: I4b84638fbb10c2b6f8f4b555e1475b0669c2daf4 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/305148 Reviewed-by: Raman Tenneti Tested-by: Mike Frysinger --- subcmds/sync.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/subcmds/sync.py b/subcmds/sync.py index b50fd94d..13ae0fed 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -801,7 +801,7 @@ later is required to fix a server side protocol bug. start, time.time(), clean) if not clean: sys.exit(1) - self._ReloadManifest(opt.manifest_name) + self._ReloadManifest(manifest_name) if opt.jobs is None: self.jobs = self.manifest.default.sync_j