Re: [gentoo-portage-dev] [PATCH] SyncScheduler: only sync the selected masters

[Brian Dolbec]

On Sun, 16 Aug 2015 00:15:07 -0700
Zac Medico <zmedico@g.o> wrote:

> Since commit 496ff326dc18890889d1ea5d2aec590394635960, master
> repositories have been synced even when not selected.
>
> Fixes: 496ff326dc18 ("sync repositories in parallel (bug 557426)")
> ---
>  pym/portage/emaint/modules/sync/sync.py | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/pym/portage/emaint/modules/sync/sync.py
> b/pym/portage/emaint/modules/sync/sync.py index 879d0f0..601118f
> 100644 --- a/pym/portage/emaint/modules/sync/sync.py
> +++ b/pym/portage/emaint/modules/sync/sync.py
> @@ -328,12 +328,15 @@ class SyncScheduler(AsyncScheduler):
>  		self._leaf_nodes = []
>  		self._repo_map = {}
>  		self._running_repos = set()
> +		selected_repo_names = frozenset(repo.name
> +			for repo in self._selected_repos)
>  		for repo in self._selected_repos:
>  			self._repo_map[repo.name] = repo
>  			self._sync_graph.add(repo.name, None)
>  			for master in repo.masters:
> -				self._repo_map[master.name] = master
> -				self._sync_graph.add(master.name,
> repo.name)
> +				if master.name in
> selected_repo_names:
> +					self._repo_map[master.name]
> = master
> +
> self._sync_graph.add(master.name, repo.name) self._update_leaf_nodes()
>
>  	def _task_exit(self, task):

looks fine

-- 
Brian Dolbec <dolsen>