Refactoring: Extract interface for VC in release scripts.

Refactoring: Extract interface for VC in release scripts.

Also simplify tagging in merge_to_branch.

BUG=chromium:410721
LOG=n
R=jkummerow@chromium.org, tandrii@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=24228

[Michael Achenbach, 2014-09-25 11:34:13]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org, tandrii@chromium.org

[Michael Achenbach, 2014-09-25 11:35:00]

PTAL. jkummerow for the changed tagging method in merge_to_branch.

[tandrii(chromium), 2014-09-25 12:50:08]

lgtm

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/common_i...
File tools/push-to-trunk/common_includes.py (right):

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/common_i...
tools/push-to-trunk/common_includes.py:263: raise Exception()
any reason why not
raise NotImplementedError

[Jakob Kummerow, 2014-09-25 12:58:17]

lgtm

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/merge_to...
File tools/push-to-trunk/merge_to_branch.py (left):

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/merge_to...
tools/push-to-trunk/merge_to_branch.py:257: print "svn revision: %s" %
self["svn_revision"]
It would be nice to keep printing the commit where the backmerge was landed, if
it's not too much trouble. Saves a manual lookup when you want to proceed to
roll it somewhere, or tell someone about it.

[Michael Achenbach, 2014-09-25 13:13:48]

Done.

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/common_i...
File tools/push-to-trunk/common_includes.py (right):

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/common_i...
tools/push-to-trunk/common_includes.py:263: raise Exception()
On 2014/09/25 12:50:08, tandrii wrote:
> any reason why not
> raise NotImplementedError

Done.

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/merge_to...
File tools/push-to-trunk/merge_to_branch.py (left):

https://codereview.chromium.org/607463002/diff/1/tools/push-to-trunk/merge_to...
tools/push-to-trunk/merge_to_branch.py:257: print "svn revision: %s" %
self["svn_revision"]
On 2014/09/25 12:58:17, Jakob wrote:
> It would be nice to keep printing the commit where the backmerge was landed,
if
> it's not too much trouble. Saves a manual lookup when you want to proceed to
> roll it somewhere, or tell someone about it.

If printing was the only reason for this code, I'd rather delete it for now and
recreate it after the git switch to make things easier.

[Michael Achenbach, 2014-09-25 13:25:26]

Committed patchset #2 (id:20001) manually as 24228 (tree was closed).