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

Issue 185263003: Refactoring: Make script dependencies more object-oriented in push and merge scripts. (Closed)

Created:
6 years, 9 months ago by Michael Achenbach
Modified:
6 years, 9 months ago
Reviewers:
ulan
CC:
v8-dev
Visibility:
Public.

Description

Refactoring: Make script dependencies more object-oriented in push and merge scripts. - Connect the top-level scripts via inheritance and remove top-level functions - Options and option processing can be extended from base to subclass script This CL is split off from https://codereview.chromium.org/173983002/ BUG= R=ulan@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=19664

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+255 lines, -339 lines) Patch
M tools/push-to-trunk/auto_roll.py View 4 chunks +35 lines, -62 lines 0 comments Download
M tools/push-to-trunk/common_includes.py View 4 chunks +79 lines, -28 lines 0 comments Download
M tools/push-to-trunk/merge_to_branch.py View 2 chunks +54 lines, -86 lines 0 comments Download
M tools/push-to-trunk/push_to_trunk.py View 2 chunks +62 lines, -114 lines 0 comments Download
M tools/push-to-trunk/test_scripts.py View 13 chunks +25 lines, -49 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Achenbach
PTAL
6 years, 9 months ago (2014-03-02 19:55:07 UTC) #1
ulan
lgtm
6 years, 9 months ago (2014-03-04 10:48:56 UTC) #2
Michael Achenbach
6 years, 9 months ago (2014-03-04 23:27:34 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r19664 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698