From: David James <davidjames@google.com>
Date: Mon, 11 Apr 2011 19:00:39 +0000 (-0700)
Subject: Remove unnecessary sync in Scheduler.py.
X-Git-Tag: v2.2.0_alpha30~6
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=1caaa48d23b1d39ee2e8e0632a08cceeba6a6b3a;p=portage.git

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 55febf745..dfccbc407 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):