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

Issue 6250177: Move reverting code from gclient_scm to scm to be able to reuse the code. (Closed)

Created:
9 years, 10 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, M-A Ruel, Peter Mayo (wrong one)
Visibility:
Public.

Description

Move reverting code from gclient_scm to scm to be able to reuse the code. There is 2 changes while moving the code: - Making externals handling an option. - Not trapping EnvironmentError. This code will be reused by the commit queue. TEST=unit tests BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=74181

Patch Set 1 #

Patch Set 2 : Don't trap weirdo exceptions #

Patch Set 3 : Fix presubmit check #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -37 lines) Patch
M gclient_scm.py View 1 1 chunk +7 lines, -34 lines 0 comments Download
M presubmit_canned_checks.py View 1 2 1 chunk +1 line, -1 line 0 comments Download
M scm.py View 1 2 chunks +46 lines, -0 lines 0 comments Download
M tests/scm_unittest.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
9 years, 10 months ago (2011-02-05 03:22:21 UTC) #1
Dirk Pranke
9 years, 10 months ago (2011-02-07 20:30:54 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698