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

Issue 3168012: Don't pass on --jobs to individual emerge processes 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

Don't pass on --jobs to individual emerge processes in parallel_emerge. When we pass on --jobs to individual emerge processes, it causes them to hide their output. This makes it harder to debug why they are failing or hanging. Example: http://goo.gl/C6Ru TEST=Made sure that output is shown by emerge processes when --jobs is used. BUG=chromium-os:5684

Patch Set 1 #

Total comments: 1

Patch Set 2 : Add comments because Nick loves comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
M parallel_emerge View 1 1 chunk +5 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
davidjames
10 years, 4 months ago (2010-08-13 17:47:59 UTC) #1
Nick Sanders
10 years, 4 months ago (2010-08-13 20:29:38 UTC) #2
lgtm w/ comment

http://codereview.chromium.org/3168012/diff/1/2
File parallel_emerge (right):

http://codereview.chromium.org/3168012/diff/1/2#newcode1298
parallel_emerge:1298: emerge.opts.pop("--jobs", multiprocessing.cpu_count()))
can you add a comment here

Powered by Google App Engine
This is Rietveld 408576698