Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(675)

Issue 6813084: Remove unnecessary sync in Scheduler.py. (Closed)

Created:
9 years, 8 months ago by davidjames
Modified:
9 years, 4 months ago
Reviewers:
zmedico, anush
CC:
chromium-os-reviews_chromium.org
Base URL:
git://git.overlays.gentoo.org/proj/portage.git@master
Visibility:
Public.

Description

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 R=zmedico@gmail.com,anush@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M pym/_emerge/Scheduler.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
9 years, 8 months ago (2011-04-11 18:17:05 UTC) #1
zmedico
9 years, 8 months ago (2011-04-11 19:04:37 UTC) #2

Powered by Google App Engine
This is Rietveld 408576698