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

Issue 399009: Reapply 32057, 32058, 32059, 32062 and fixes problems introduced by these changes. (Closed)

Created:
11 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Reapply 32057, 32058, 32059, 32062 and fixes problems introduced by these changes. Noteworthy change is scm.SVN.GetFileProperty calls Capture instead of Run. TEST=unit tests BUG=none

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+699 lines, -680 lines) Patch
M gcl.py View 12 chunks +15 lines, -45 lines 0 comments Download
M gclient.py View 3 chunks +4 lines, -4 lines 0 comments Download
M gclient_scm.py View 19 chunks +47 lines, -62 lines 0 comments Download
M gclient_utils.py View 7 chunks +11 lines, -11 lines 0 comments Download
M git_cl_hooks.py View 2 chunks +2 lines, -3 lines 0 comments Download
M presubmit_support.py View 13 chunks +17 lines, -17 lines 0 comments Download
M revert.py View 1 chunk +1 line, -1 line 0 comments Download
M scm.py View 1 chunk +401 lines, -322 lines 0 comments Download
M tests/gcl_unittest.py View 4 chunks +20 lines, -27 lines 0 comments Download
M tests/gclient_scm_test.py View 11 chunks +43 lines, -33 lines 0 comments Download
M tests/gclient_test.py View 2 chunks +5 lines, -12 lines 0 comments Download
M tests/presubmit_unittest.py View 21 chunks +65 lines, -59 lines 0 comments Download
M tests/revert_unittest.py View 2 chunks +2 lines, -1 line 0 comments Download
M tests/scm_unittest.py View 11 chunks +57 lines, -33 lines 0 comments Download
M tests/super_mox.py View 1 chunk +1 line, -1 line 0 comments Download
M tests/trychange_unittest.py View 1 chunk +3 lines, -3 lines 0 comments Download
M trychange.py View 3 chunks +5 lines, -46 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
11 years, 1 month ago (2009-11-16 20:31:19 UTC) #1
Dirk Pranke
11 years, 1 month ago (2009-11-17 01:39:17 UTC) #2
lgtm more or less as an un-revert; I can't speak to the correctness of the fix.

Powered by Google App Engine
This is Rietveld 408576698