Refactoring: Redesign option parsing in push and merge scripts.

Refactoring: Redesign option parsing in push and merge scripts.

- Deprecate optparse with argparse
- Connect the top-level scripts via inheritance and remove top-level functions
- Options and option processing can be extended from base to subclass script
- All options have now long names
- The tests include now options parsing by default: each test specifies the command-line args to parse rather than the options directly

[Michael Achenbach, 2014-02-21 13:57:11]

PTAL

[tfarina, 2014-02-21 17:03:18]

are these scripts used by buildbot? Do they have python 2.7? argparse may not be
available there if they have only python 2.6. This happened in skia.

[Michael Achenbach, 2014-02-21 20:43:00]

No - these are all scripts use by developers locally only. Should be no problem.

[tfarina, 2014-02-24 01:20:18]

Moving myself to CC list as I'm not reviewing this. Removed Ravi as well.

[ulan, 2014-02-24 07:36:30]

On 2014/02/24 01:20:18, tfarina wrote:
> Moving myself to CC list as I'm not reviewing this. Removed Ravi as well.

> - Deprecate optparse with argparse 
> - Connect the top-level scripts via inheritance and remove top-level functions

> - Options and option processing can be extended from base to subclass script 
> - All options have now long names 
> - The tests include now options parsing by default: each test specifies the
command-line args to parse rather than the options directly
Any reason why this is one big CL instead of five small CLs?

[Michael Achenbach, 2014-02-24 15:47:36]

On 2014/02/24 07:36:30, ulan wrote:
> On 2014/02/24 01:20:18, tfarina wrote:
> > Moving myself to CC list as I'm not reviewing this. Removed Ravi as well.
> 
> > - Deprecate optparse with argparse 
> > - Connect the top-level scripts via inheritance and remove top-level
functions
> 
> > - Options and option processing can be extended from base to subclass script

> > - All options have now long names 
> > - The tests include now options parsing by default: each test specifies the
> command-line args to parse rather than the options directly
> Any reason why this is one big CL instead of five small CLs?

I'll try to split in in smaller chunks...