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

Issue 5968005: Move patch management functions into their own file, patch.py. (Closed)

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

Description

Move patch management functions into their own file, patch.py. This is in preparation to remove svn_utils.py altogether. BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=/70060

Patch Set 1 #

Patch Set 2 : rebase against trunk #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -39 lines) Patch
M checkout.py View 6 chunks +11 lines, -6 lines 0 comments Download
A patch.py View 1 chunk +31 lines, -0 lines 0 comments Download
M pending_manager.py View 1 3 chunks +4 lines, -17 lines 0 comments Download
M svn_utils.py View 1 1 chunk +0 lines, -9 lines 0 comments Download
M tests/checkout_test.py View 1 4 chunks +4 lines, -4 lines 0 comments Download
M tests/svn_utils_test.py View 1 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
Dependent on http://codereview.chromium.org/6072006.
10 years ago (2010-12-22 17:04:31 UTC) #1
Dirk Pranke
LGTM.
10 years ago (2010-12-22 23:05:20 UTC) #2
commit-bot: I haz the power
10 years ago (2010-12-23 16:01:35 UTC) #3
Patch failed to apply against 70059.

Powered by Google App Engine
This is Rietveld 408576698