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

Issue 155759: Add a --nohooks to gclient to update/sync/revert without calling the hooks (Closed)

Created:
11 years, 5 months ago by fta
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Add a --nohooks to gclient to update/sync/revert without calling the hooks BUG=16856

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -4 lines) Patch
M gclient.py View 5 chunks +14 lines, -3 lines 1 comment Download
M tests/gclient_test.py View 3 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
Nicolas Sylvain
LGTM with the nit. I'm not sure about the exact process for submitting this. It ...
11 years, 5 months ago (2009-07-20 23:03:19 UTC) #1
M-A Ruel
lgtm, you don't need parenthesis for multiline strings as parameter.
11 years, 5 months ago (2009-07-21 10:39:05 UTC) #2
Evan Martin
are you gonna commit it? :)
11 years, 5 months ago (2009-07-25 01:22:22 UTC) #3
Nicolas Sylvain
Do it ;) Nicolas On Fri, Jul 24, 2009 at 6:22 PM, <evan@chromium.org> wrote: > ...
11 years, 5 months ago (2009-07-26 20:48:44 UTC) #4
Evan Martin
Ohhhh, fta wrote it. I was confused.
11 years, 5 months ago (2009-07-27 15:41:47 UTC) #5
Evan Martin
On 2009/07/27 15:41:47, Evan Martin wrote: > Ohhhh, fta wrote it. I was confused. I ...
11 years, 5 months ago (2009-07-27 15:44:32 UTC) #6
Nicolas Sylvain
Why would you commit to src.chromium.org? this is read only :) Nicolas On Mon, Jul ...
11 years, 5 months ago (2009-07-27 17:15:47 UTC) #7
Evan Martin
I always get confused between {http,svn}://{src,svn}...
11 years, 5 months ago (2009-07-27 17:23:19 UTC) #8
Evan Martin
in as r21655
11 years, 5 months ago (2009-07-27 17:23:28 UTC) #9
Nicolas Sylvain
11 years, 5 months ago (2009-07-27 17:54:39 UTC) #10
thanks!

On Mon, Jul 27, 2009 at 10:23 AM, <evan@chromium.org> wrote:

> in as r21655
>
>
> http://codereview.chromium.org/155759
>

Powered by Google App Engine
This is Rietveld 408576698