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

Issue 15421005: Don't "hang" gclient if fast-forward is not possible. (Closed)

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

Description

Don't "hang" gclient if fast-forward is not possible. As noticed in the TODO, raw_input (used by ask_for_data) works badly on background threads. On my Windows it doesn't even print the prompt which makes it hard to interact with it. Better to just abort and tell the user of gclient how to proceed. BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=201295

Patch Set 1 #

Patch Set 2 : Generalizing. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -5 lines) Patch
M gclient_scm.py View 1 3 chunks +8 lines, -5 lines 0 comments Download
M tests/gclient_scm_test.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Daniel Bratell
Better to abort than to hang.
7 years, 7 months ago (2013-05-20 12:09:45 UTC) #1
M-A Ruel
lgtm
7 years, 7 months ago (2013-05-20 20:19:09 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/15421005/3001
7 years, 7 months ago (2013-05-21 13:29:14 UTC) #3
commit-bot: I haz the power
Change committed as 201295
7 years, 7 months ago (2013-05-21 13:30:47 UTC) #4
droger
Is it still possible to use gclient in multithreaded mode? Previously I was able to ...
7 years, 6 months ago (2013-06-03 11:27:50 UTC) #5
M-A Ruel
"simply fails" is not informative. M-A 2013/6/3 <droger@chromium.org> > Is it still possible to use ...
7 years, 6 months ago (2013-06-03 11:36:14 UTC) #6
Daniel Bratell
On 2013/06/03 11:27:50, droger wrote: > Now the scripts simply fails, and I have to ...
7 years, 6 months ago (2013-06-03 12:02:22 UTC) #7
M-A Ruel
7 years, 6 months ago (2013-06-03 12:06:25 UTC) #8
Message was sent while issue was closed.
On 2013/06/03 12:02:22, bratell wrote:
> On 2013/06/03 11:27:50, droger wrote:
> > Now the scripts simply fails, and I have to run it single threaded, which is
> > significantly slower.
> 
> I don't know in what way it fails, but if it's about aborting waiting for
> feedback, I guess you could do single-thread, respond to the question(s), then
> abort and do multi-process, though I agree with M-A that it's hard to say
> without knowing exactly what goes wrong.

Also note https://codereview.chromium.org/16255002/ could (?) be helpful.

Powered by Google App Engine
This is Rietveld 408576698