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

Issue 110883004: Avoid windows presubmit hang in gclient tests. (Closed)

Created:
7 years ago by Daniel Bratell
Modified:
7 years ago
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Visibility:
Public.

Description

Avoid windows presubmit hang in gclient tests. If os.kill(os.getpid(), signal.CTRL_C_EVENT) doesn't do what the test author hopes it will do, the test will hang until it has calculated a million out-of-thread/process results. That is approximately an infinite hang. BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=240319

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M tests/git_common_test.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Daniel Bratell
Just avoiding a test failure turning into a hang. (Soon it will at least be ...
7 years ago (2013-12-10 14:44:52 UTC) #1
M-A Ruel
thanks, lgtm
7 years ago (2013-12-12 14:13:06 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/bratell@opera.com/110883004/1
7 years ago (2013-12-12 14:13:49 UTC) #3
commit-bot: I haz the power
7 years ago (2013-12-12 14:16:51 UTC) #4
Message was sent while issue was closed.
Change committed as 240319

Powered by Google App Engine
This is Rietveld 408576698