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

diff --git a/pym/_emerge/Scheduler.py b/pym/_emerge/Scheduler.py
index 55febf745826a7323232b1234b4bd36663201889..dfccbc40768068e2ff8d86eec8be95d65262f009 100644 (file)
--- 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):