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

Issue 3184011: Cleanup cycle cracking in parallel_emerge. (Closed)

Created:
10 years, 4 months ago by davidjames
Modified:
9 years, 4 months ago
Reviewers:
Nick Sanders
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Base URL:
ssh://git@chromiumos-git/crosutils.git
Visibility:
Public.

Description

Cleanup cycle cracking in parallel_emerge. I've cleaned up the cycle cracking in parallel_emerge to hopefully be significantly easier to understand. This also fixes some bugs. Most significantly, this patch restores the dependency cracking so that it actually cracks cycles that are against the emerge ordering, instead of cracking everything but those dependencies. This was only broken temporarily due to my fix for Bug 5795. Oops. Fortunately, this bug doesn't affect many people, because make_chroot --fast is deprecated anyway. TEST=make_chroot --fast, parallel_emerge -pe hard-host-depends BUG=chromium-os:5795

Patch Set 1 #

Patch Set 2 : 80 chars #

Patch Set 3 : No changes (hopefully) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -37 lines) Patch
M parallel_emerge View 1 3 chunks +54 lines, -37 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
Cycle cracking should be a lot cleaner now, and actually finally work properly I hope ...
10 years, 4 months ago (2010-08-17 22:41:29 UTC) #1
Nick Sanders
10 years, 4 months ago (2010-08-20 03:35:27 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698