From f7fbe17d589f5d404131e41cff55a68e4fb19df6 Mon Sep 17 00:00:00 2001 From: David James Date: Mon, 11 Apr 2011 12:00:39 -0700 Subject: [PATCH] Remove unnecessary sync in Scheduler.py. If we are passed a fakevartree object, it should already be sync'd, so there is no need to sync it again here. This sync is only needed when we create a new FakeVartree, so I moved the sync to the right place to fix this. BUG=chromium-os:14035 TEST=Build a bunch of packages with this patch. Change-Id: I89d79cf946f4c0c27ad585ad7c88a41985260342 Review URL: http://codereview.chromium.org/6813084 --- pym/_emerge/Scheduler.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pym/_emerge/Scheduler.py b/pym/_emerge/Scheduler.py index c9b7c9fad..abb4accb8 100644 --- a/pym/_emerge/Scheduler.py +++ b/pym/_emerge/Scheduler.py @@ -323,9 +323,9 @@ class Scheduler(PollScheduler): if graph_config is None: fake_vartree = FakeVartree(self.trees[root]["root_config"], pkg_cache=self._pkg_cache) + fake_vartree.sync() else: fake_vartree = graph_config.trees[root]['vartree'] - fake_vartree.sync() self._blocker_db[root] = BlockerDB(fake_vartree) def _destroy_graph(self): -- 2.26.2