bisect-builds.py: Use the continuous archive, rather than the snapshots.

bisect-builds.py: Use the continuous archive, rather than the snapshots.

This will allow people to bisect back until 2008 on Windows and 2009 for Linux
and Mac, rather than just the last ~1000 revisions.

BUG=none
TEST=bisect-builds.py works as before, but with a larger archive history

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=83774

[Evan Martin, 2011-04-14 19:03:56]

LGTM

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py
File tools/bisect-builds.py (left):

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#oldcode86
tools/bisect-builds.py:86: 
fwiw, double-newline after function bodies is recommended python style, so i
suggest not removing this line

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py
File tools/bisect-builds.py (right):

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#newcode150
tools/bisect-builds.py:150: revlist = filter(lambda r:
revrange.__contains__(GetRevision(r)),
GetRevision(r) in revrange

?

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#newcode219
tools/bisect-builds.py:219: choices = ['mac', 'xp', 'linux', 'linux64']#,
'linux-chromiumos'] http://crbug.com/78158
typo here, extra pasted url

[Robert Sesek, 2011-04-29 20:25:44]

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py
File tools/bisect-builds.py (left):

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#oldcode86
tools/bisect-builds.py:86: 
On 2011/04/14 19:03:56, Evan Martin wrote:
> fwiw, double-newline after function bodies is recommended python style, so i
> suggest not removing this line

Done.

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py
File tools/bisect-builds.py (right):

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#newcode150
tools/bisect-builds.py:150: revlist = filter(lambda r:
revrange.__contains__(GetRevision(r)),
On 2011/04/14 19:03:56, Evan Martin wrote:
> GetRevision(r) in revrange
> 
> ?

Done.

http://codereview.chromium.org/6788015/diff/1/tools/bisect-builds.py#newcode219
tools/bisect-builds.py:219: choices = ['mac', 'xp', 'linux', 'linux64']#,
'linux-chromiumos'] http://crbug.com/78158
On 2011/04/14 19:03:56, Evan Martin wrote:
> typo here, extra pasted url

No, it's actually a bug that chromiumos doesn't have a continuous archive.
Cleaned up a bit.

[Evan Martin, 2011-05-02 18:29:17]

LGTM

http://codereview.chromium.org/6788015/diff/4002/tools/bisect-builds.py
File tools/bisect-builds.py (right):

http://codereview.chromium.org/6788015/diff/4002/tools/bisect-builds.py#newco...
tools/bisect-builds.py:26: # can be obtained through ParseIndexLine().
Might be more maintainable to use something like
  /%(date)s/%(revision)d/

shrug

http://codereview.chromium.org/6788015/diff/4002/tools/bisect-builds.py#newco...
tools/bisect-builds.py:228: # 'linux-chromiumos' lacks a continuous archive
http://crbug.com/78158
80 cols