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

Issue 4465001: Reduce the chances for gclient to throw an exception in subprocess module. (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
CC:
chromium-reviews, M-A Ruel
Visibility:
Public.

Description

Reduce the chances for gclient to throw an exception in subprocess module. Python standard library is not thread safe. Work around it by leaking a bit. The problem is particularly visible on python 2.4. TEST=none BUG=60623 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=65048

Patch Set 1 #

Patch Set 2 : fix license and unit test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -15 lines) Patch
M gclient_utils.py View 1 3 chunks +12 lines, -13 lines 0 comments Download
M tests/gclient_utils_test.py View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
10 years, 1 month ago (2010-11-04 01:58:00 UTC) #1
jochen (gone - plz use gerrit)
10 years, 1 month ago (2010-11-04 08:05:50 UTC) #2
lgtm (although it actually more looks like a hack to me...)

btw, why does this file have a different license?

Powered by Google App Engine
This is Rietveld 408576698