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

Issue 213002: Fixing errors in gclient on a gclient revert --nohooks on windows.... (Closed)

Created:
11 years, 3 months ago by bradn
Modified:
9 years, 7 months ago
Reviewers:
M-A Ruel
CC:
chromium-reviews_googlegroups.com, Mandeep Singh Baines
Visibility:
Public.

Description

Fixing errors in gclient on a gclient revert --nohooks on windows. BUG=None TEST=None TBR=maruel Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=26425

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M gclient_scm.py View 2 chunks +2 lines, -2 lines 0 comments Download
M gclient_utils.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
bradn
11 years, 3 months ago (2009-09-17 00:48:15 UTC) #1
M-A Ruel
Thanks Mandeep, you now need to write unit test, we can't afford to break gclient ...
11 years, 3 months ago (2009-09-17 00:50:00 UTC) #2
msb
11 years, 3 months ago (2009-09-17 04:29:58 UTC) #3
maruel@chromium.org (maruel@chromium.org) wrote:
> Thanks
>
> Mandeep, you now need to write unit test, we can't afford to break gclient
> further.
>

Sorry about the breakage:(

Unit test review here:

http://codereview.chromium.org/214005

Nearly done with the refactoring. Majority of my future changes will only
add new git functionality so future changes should be less invasive. I will
add a new unit test for any new code which isn't adaquately tested by the
existing unit tests.

> http://codereview.chromium.org/213002

Powered by Google App Engine
This is Rietveld 408576698