Index: tools/bisect-builds.py |
diff --git a/tools/bisect-builds.py b/tools/bisect-builds.py |
index 75237d5b521f73e8c9514cb1836b5cc696ec1254..a7938dfa1f9c8049d7597bfbf3d42a3106a54e7c 100755 |
--- a/tools/bisect-builds.py |
+++ b/tools/bisect-builds.py |
@@ -93,7 +93,7 @@ class PathContext(object): |
"""A PathContext is used to carry the information used to construct URLs and |
paths when dealing with the storage server and archives.""" |
def __init__(self, base_url, platform, good_revision, bad_revision, |
- is_official, is_aura, is_asan, use_local_repo, flash_path = None, |
+ is_official, is_asan, use_local_repo, flash_path = None, |
pdf_path = None): |
super(PathContext, self).__init__() |
# Store off the input parameters. |
@@ -102,7 +102,6 @@ class PathContext(object): |
self.good_revision = good_revision |
self.bad_revision = bad_revision |
self.is_official = is_official |
- self.is_aura = is_aura |
self.is_asan = is_asan |
self.build_type = 'release' |
self.flash_path = flash_path |
@@ -151,10 +150,7 @@ class PathContext(object): |
self._listing_platform_dir = 'mac/' |
self._binary_name = 'Google Chrome.app/Contents/MacOS/Google Chrome' |
elif self.platform == 'win': |
- if self.is_aura: |
- self._listing_platform_dir = 'win-aura/' |
- else: |
- self._listing_platform_dir = 'win/' |
+ self._listing_platform_dir = 'win/' |
else: |
if self.platform in ('linux', 'linux64', 'linux-arm'): |
self.archive_name = 'chrome-linux.zip' |
@@ -229,16 +225,6 @@ class PathContext(object): |
extract_dir = self._archive_extract_dir |
return os.path.join(extract_dir, self._binary_name) |
- @staticmethod |
- def IsAuraBuild(build): |
- """Checks whether the given build is an Aura build.""" |
- return build.split('.')[3] == '1' |
- |
- @staticmethod |
- def IsOfficialASANBuild(build): |
- """Checks whether the given build is an ASAN build.""" |
- return build.split('.')[3] == '2' |
- |
def ParseDirectoryIndex(self): |
"""Parses the Google Storage directory listing into a list of revision |
numbers.""" |
@@ -427,17 +413,7 @@ class PathContext(object): |
if build_number > maxrev: |
break |
if build_number >= minrev: |
- # If we are bisecting Aura, we want to include only builds which |
- # ends with ".1". |
- if self.is_aura: |
- if self.IsAuraBuild(str(build_number)): |
- final_list.append(str(build_number)) |
- # If we are bisecting only official builds (without --aura), |
- # we can not include builds which ends with '.1' or '.2' since |
- # they have different folder hierarchy inside. |
- elif (not self.IsAuraBuild(str(build_number)) and |
- not self.IsOfficialASANBuild(str(build_number))): |
- final_list.append(str(build_number)) |
+ final_list.append(str(build_number)) |
except urllib.HTTPError: |
pass |
return final_list |
@@ -991,11 +967,6 @@ def main(): |
action='store_true', |
default=False, |
help='Use command exit code to tell good/bad revision.') |
- parser.add_option('--aura', |
- dest='aura', |
- action='store_true', |
- default=False, |
- help='Allow the script to bisect aura builds') |
parser.add_option('--asan', |
dest='asan', |
action='store_true', |
@@ -1017,12 +988,6 @@ def main(): |
parser.print_help() |
return 1 |
- if opts.aura: |
- if opts.archive != 'win' or not opts.official_builds: |
- print ('Error: Aura is supported only on Windows platform ' |
- 'and official builds.') |
- return 1 |
- |
if opts.asan: |
supported_platforms = ['linux', 'mac', 'win'] |
if opts.archive not in supported_platforms: |
@@ -1042,8 +1007,8 @@ def main(): |
# Create the context. Initialize 0 for the revisions as they are set below. |
context = PathContext(base_url, opts.archive, opts.good, opts.bad, |
- opts.official_builds, opts.aura, opts.asan, |
- opts.use_local_repo, opts.flash_path, opts.pdf_path) |
+ opts.official_builds, opts.asan, opts.use_local_repo, |
+ opts.flash_path, opts.pdf_path) |
# Pick a starting point, try to get HEAD for this. |
if not opts.bad: |
context.bad_revision = '999.0.0.0' |