blob: 9efc7bfa8fa33eaa0ced331de4833a4e559b5ab6 [file] [log] [blame]
[email protected]cb155a82011-11-29 17:25:341#!/usr/bin/env python
[email protected]5e93cf162012-01-28 02:16:562# Copyright (c) 2012 The Chromium Authors. All rights reserved.
[email protected]67e0bc62009-09-03 22:06:093# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
5
6"""Snapshot Build Bisect Tool
7
[email protected]7ad66a72009-09-04 17:52:338This script bisects a snapshot archive using binary search. It starts at
[email protected]67e0bc62009-09-03 22:06:099a bad revision (it will try to guess HEAD) and asks for a last known-good
10revision. It will then binary search across this revision range by downloading,
11unzipping, and opening Chromium for you. After testing the specific revision,
12it will ask you whether it is good or bad before continuing the search.
[email protected]67e0bc62009-09-03 22:06:0913"""
14
Raul Tambre57e09d62019-09-22 17:18:5215from __future__ import print_function
16
[email protected]4df583c2014-07-31 17:11:5517# The base URL for stored build archives.
18CHROMIUM_BASE_URL = ('http://commondatastorage.googleapis.com'
19 '/chromium-browser-snapshots')
20WEBKIT_BASE_URL = ('http://commondatastorage.googleapis.com'
21 '/chromium-webkit-snapshots')
[email protected]011886692014-08-01 21:00:2122ASAN_BASE_URL = ('http://commondatastorage.googleapis.com'
23 '/chromium-browser-asan')
[email protected]67e0bc62009-09-03 22:06:0924
[email protected]4df583c2014-07-31 17:11:5525# URL template for viewing changelogs between revisions.
pshenoy9ce271f2014-09-02 22:14:0526CHANGELOG_URL = ('https://chromium.googlesource.com/chromium/src/+log/%s..%s')
27
28# URL to convert SVN revision to git hash.
pshenoy13cb79e02014-09-05 01:42:5329CRREV_URL = ('https://cr-rev.appspot.com/_ah/api/crrev/v1/redirect/')
[email protected]f6a71a72009-10-08 19:55:3830
[email protected]b2fe7f22011-10-25 22:58:3131# DEPS file URL.
Di Mu08c59682016-07-11 23:05:0732DEPS_FILE = ('https://chromium.googlesource.com/chromium/src/+/%s/DEPS')
[email protected]b2fe7f22011-10-25 22:58:3133
[email protected]4df583c2014-07-31 17:11:5534# Blink changelogs URL.
35BLINK_CHANGELOG_URL = ('http://build.chromium.org'
36 '/f/chromium/perf/dashboard/ui/changelog_blink.html'
37 '?url=/trunk&range=%d%%3A%d')
38
39DONE_MESSAGE_GOOD_MIN = ('You are probably looking for a change made after %s ('
40 'known good), but no later than %s (first known bad).')
41DONE_MESSAGE_GOOD_MAX = ('You are probably looking for a change made after %s ('
42 'known bad), but no later than %s (first known good).')
[email protected]05ff3fd2012-04-17 23:24:0643
[email protected]3e7c85322014-06-27 20:27:3644CHROMIUM_GITHASH_TO_SVN_URL = (
45 'https://chromium.googlesource.com/chromium/src/+/%s?format=json')
[email protected]4df583c2014-07-31 17:11:5546
[email protected]3e7c85322014-06-27 20:27:3647BLINK_GITHASH_TO_SVN_URL = (
48 'https://chromium.googlesource.com/chromium/blink/+/%s?format=json')
[email protected]4df583c2014-07-31 17:11:5549
50GITHASH_TO_SVN_URL = {
51 'chromium': CHROMIUM_GITHASH_TO_SVN_URL,
52 'blink': BLINK_GITHASH_TO_SVN_URL,
53}
54
55# Search pattern to be matched in the JSON output from
[email protected]3e7c85322014-06-27 20:27:3656# CHROMIUM_GITHASH_TO_SVN_URL to get the chromium revision (svn revision).
pshenoyb23a1452014-09-05 22:52:0557CHROMIUM_SEARCH_PATTERN_OLD = (
[email protected]3e7c85322014-06-27 20:27:3658 r'.*git-svn-id: svn://svn.chromium.org/chrome/trunk/src@(\d+) ')
pshenoyb23a1452014-09-05 22:52:0559CHROMIUM_SEARCH_PATTERN = (
60 r'Cr-Commit-Position: refs/heads/master@{#(\d+)}')
[email protected]4df583c2014-07-31 17:11:5561
[email protected]3e7c85322014-06-27 20:27:3662# Search pattern to be matched in the json output from
63# BLINK_GITHASH_TO_SVN_URL to get the blink revision (svn revision).
64BLINK_SEARCH_PATTERN = (
65 r'.*git-svn-id: svn://svn.chromium.org/blink/trunk@(\d+) ')
[email protected]4df583c2014-07-31 17:11:5566
67SEARCH_PATTERN = {
68 'chromium': CHROMIUM_SEARCH_PATTERN,
69 'blink': BLINK_SEARCH_PATTERN,
70}
[email protected]3e7c85322014-06-27 20:27:3671
[email protected]480369782014-08-22 20:15:5872CREDENTIAL_ERROR_MESSAGE = ('You are attempting to access protected data with '
73 'no configured credentials')
74
[email protected]67e0bc62009-09-03 22:06:0975###############################################################################
76
Dominic Mazzoni215e80b2017-11-29 20:05:2777import glob
[email protected]83048502014-08-21 16:48:4478import httplib
[email protected]4c6fec6b2013-09-17 17:44:0879import json
[email protected]7ad66a72009-09-04 17:52:3380import optparse
[email protected]67e0bc62009-09-03 22:06:0981import os
82import re
[email protected]61ea90a2013-09-26 10:17:3483import shlex
[email protected]67e0bc62009-09-03 22:06:0984import shutil
[email protected]afe30662011-07-30 01:05:5285import subprocess
[email protected]67e0bc62009-09-03 22:06:0986import sys
[email protected]7ad66a72009-09-04 17:52:3387import tempfile
[email protected]afe30662011-07-30 01:05:5288import threading
[email protected]67e0bc62009-09-03 22:06:0989import urllib
[email protected]d0149c5c2012-05-29 21:12:1190from distutils.version import LooseVersion
[email protected]183706d92011-06-10 13:06:2291from xml.etree import ElementTree
[email protected]bd8dcb92010-03-31 01:05:2492import zipfile
93
[email protected]cb155a82011-11-29 17:25:3494
[email protected]183706d92011-06-10 13:06:2295class PathContext(object):
96 """A PathContext is used to carry the information used to construct URLs and
97 paths when dealing with the storage server and archives."""
[email protected]4c6fec6b2013-09-17 17:44:0898 def __init__(self, base_url, platform, good_revision, bad_revision,
Jason Kersey97bb027a2016-05-11 20:10:4399 is_asan, use_local_cache, flash_path = None):
[email protected]183706d92011-06-10 13:06:22100 super(PathContext, self).__init__()
101 # Store off the input parameters.
[email protected]4c6fec6b2013-09-17 17:44:08102 self.base_url = base_url
[email protected]183706d92011-06-10 13:06:22103 self.platform = platform # What's passed in to the '-a/--archive' option.
104 self.good_revision = good_revision
105 self.bad_revision = bad_revision
[email protected]011886692014-08-01 21:00:21106 self.is_asan = is_asan
107 self.build_type = 'release'
[email protected]fc3702e2013-11-09 04:23:00108 self.flash_path = flash_path
[email protected]3e7c85322014-06-27 20:27:36109 # Dictionary which stores svn revision number as key and it's
110 # corresponding git hash as value. This data is populated in
111 # _FetchAndParse and used later in GetDownloadURL while downloading
112 # the build.
113 self.githash_svn_dict = {}
[email protected]183706d92011-06-10 13:06:22114 # The name of the ZIP file in a revision directory on the server.
115 self.archive_name = None
116
rob724c9062015-01-22 00:26:42117 # Whether to cache and use the list of known revisions in a local file to
118 # speed up the initialization of the script at the next run.
119 self.use_local_cache = use_local_cache
120
121 # Locate the local checkout to speed up the script by using locally stored
122 # metadata.
123 abs_file_path = os.path.abspath(os.path.realpath(__file__))
124 local_src_path = os.path.join(os.path.dirname(abs_file_path), '..')
125 if abs_file_path.endswith(os.path.join('tools', 'bisect-builds.py')) and\
126 os.path.exists(os.path.join(local_src_path, '.git')):
127 self.local_src_path = os.path.normpath(local_src_path)
128 else:
129 self.local_src_path = None
[email protected]6a7a5d62014-07-09 04:45:50130
[email protected]183706d92011-06-10 13:06:22131 # Set some internal members:
132 # _listing_platform_dir = Directory that holds revisions. Ends with a '/'.
133 # _archive_extract_dir = Uncompressed directory in the archive_name file.
134 # _binary_name = The name of the executable to run.
dmazzoni76e907d2015-01-22 08:14:49135 if self.platform in ('linux', 'linux64', 'linux-arm', 'chromeos'):
[email protected]183706d92011-06-10 13:06:22136 self._binary_name = 'chrome'
[email protected]480369782014-08-22 20:15:58137 elif self.platform in ('mac', 'mac64'):
[email protected]183706d92011-06-10 13:06:22138 self.archive_name = 'chrome-mac.zip'
139 self._archive_extract_dir = 'chrome-mac'
[email protected]480369782014-08-22 20:15:58140 elif self.platform in ('win', 'win64'):
Dominic Mazzonie84e40b2018-10-08 06:44:45141 # Note: changed at revision 591483; see GetDownloadURL and GetLaunchPath
142 # below where these are patched.
[email protected]183706d92011-06-10 13:06:22143 self.archive_name = 'chrome-win32.zip'
144 self._archive_extract_dir = 'chrome-win32'
145 self._binary_name = 'chrome.exe'
146 else:
[email protected]afe30662011-07-30 01:05:52147 raise Exception('Invalid platform: %s' % self.platform)
[email protected]183706d92011-06-10 13:06:22148
Jason Kersey97bb027a2016-05-11 20:10:43149 if self.platform in ('linux', 'linux64', 'linux-arm', 'chromeos'):
Dominic Mazzonie84e40b2018-10-08 06:44:45150 # Note: changed at revision 591483; see GetDownloadURL and GetLaunchPath
151 # below where these are patched.
Jason Kersey97bb027a2016-05-11 20:10:43152 self.archive_name = 'chrome-linux.zip'
153 self._archive_extract_dir = 'chrome-linux'
[email protected]d0149c5c2012-05-29 21:12:11154 if self.platform == 'linux':
Jason Kersey97bb027a2016-05-11 20:10:43155 self._listing_platform_dir = 'Linux/'
[email protected]d0149c5c2012-05-29 21:12:11156 elif self.platform == 'linux64':
Jason Kersey97bb027a2016-05-11 20:10:43157 self._listing_platform_dir = 'Linux_x64/'
158 elif self.platform == 'linux-arm':
159 self._listing_platform_dir = 'Linux_ARM_Cross-Compile/'
160 elif self.platform == 'chromeos':
161 self._listing_platform_dir = 'Linux_ChromiumOS_Full/'
162 elif self.platform in ('mac', 'mac64'):
163 self._listing_platform_dir = 'Mac/'
164 self._binary_name = 'Chromium.app/Contents/MacOS/Chromium'
165 elif self.platform == 'win':
166 self._listing_platform_dir = 'Win/'
jiawei.shao734efbc92016-09-23 02:11:45167 elif self.platform == 'win64':
168 self._listing_platform_dir = 'Win_x64/'
[email protected]d0149c5c2012-05-29 21:12:11169
[email protected]011886692014-08-01 21:00:21170 def GetASANPlatformDir(self):
171 """ASAN builds are in directories like "linux-release", or have filenames
172 like "asan-win32-release-277079.zip". This aligns to our platform names
173 except in the case of Windows where they use "win32" instead of "win"."""
174 if self.platform == 'win':
175 return 'win32'
176 else:
177 return self.platform
178
[email protected]183706d92011-06-10 13:06:22179 def GetListingURL(self, marker=None):
180 """Returns the URL for a directory listing, with an optional marker."""
181 marker_param = ''
182 if marker:
183 marker_param = '&marker=' + str(marker)
[email protected]011886692014-08-01 21:00:21184 if self.is_asan:
185 prefix = '%s-%s' % (self.GetASANPlatformDir(), self.build_type)
186 return self.base_url + '/?delimiter=&prefix=' + prefix + marker_param
187 else:
188 return (self.base_url + '/?delimiter=/&prefix=' +
189 self._listing_platform_dir + marker_param)
[email protected]183706d92011-06-10 13:06:22190
191 def GetDownloadURL(self, revision):
192 """Gets the download URL for a build archive of a specific revision."""
[email protected]011886692014-08-01 21:00:21193 if self.is_asan:
194 return '%s/%s-%s/%s-%d.zip' % (
195 ASAN_BASE_URL, self.GetASANPlatformDir(), self.build_type,
196 self.GetASANBaseName(), revision)
Jason Kersey97bb027a2016-05-11 20:10:43197 if str(revision) in self.githash_svn_dict:
198 revision = self.githash_svn_dict[str(revision)]
Dominic Mazzonie84e40b2018-10-08 06:44:45199 archive_name = self.archive_name
200
201 # At revision 591483, the names of two of the archives changed
202 # due to: https://chromium-review.googlesource.com/#/q/1226086
203 # See: http://crbug.com/789612
204 if revision >= 591483:
205 if self.platform == 'chromeos':
206 archive_name = 'chrome-chromeos.zip'
207 elif self.platform in ('win', 'win64'):
208 archive_name = 'chrome-win.zip'
209
Jason Kersey97bb027a2016-05-11 20:10:43210 return '%s/%s%s/%s' % (self.base_url, self._listing_platform_dir,
Dominic Mazzonie84e40b2018-10-08 06:44:45211 revision, archive_name)
[email protected]183706d92011-06-10 13:06:22212
213 def GetLastChangeURL(self):
214 """Returns a URL to the LAST_CHANGE file."""
[email protected]4c6fec6b2013-09-17 17:44:08215 return self.base_url + '/' + self._listing_platform_dir + 'LAST_CHANGE'
[email protected]183706d92011-06-10 13:06:22216
[email protected]011886692014-08-01 21:00:21217 def GetASANBaseName(self):
218 """Returns the base name of the ASAN zip file."""
219 if 'linux' in self.platform:
220 return 'asan-symbolized-%s-%s' % (self.GetASANPlatformDir(),
221 self.build_type)
222 else:
223 return 'asan-%s-%s' % (self.GetASANPlatformDir(), self.build_type)
224
225 def GetLaunchPath(self, revision):
[email protected]183706d92011-06-10 13:06:22226 """Returns a relative path (presumably from the archive extraction location)
227 that is used to run the executable."""
[email protected]011886692014-08-01 21:00:21228 if self.is_asan:
229 extract_dir = '%s-%d' % (self.GetASANBaseName(), revision)
230 else:
231 extract_dir = self._archive_extract_dir
Dominic Mazzonie84e40b2018-10-08 06:44:45232
233 # At revision 591483, the names of two of the archives changed
234 # due to: https://chromium-review.googlesource.com/#/q/1226086
235 # See: http://crbug.com/789612
236 if revision >= 591483:
237 if self.platform == 'chromeos':
238 extract_dir = 'chrome-chromeos'
239 elif self.platform in ('win', 'win64'):
Lei Zhang1c8c6f7e2018-11-09 16:46:30240 extract_dir = 'chrome-win'
Dominic Mazzonie84e40b2018-10-08 06:44:45241
[email protected]011886692014-08-01 21:00:21242 return os.path.join(extract_dir, self._binary_name)
[email protected]183706d92011-06-10 13:06:22243
rob724c9062015-01-22 00:26:42244 def ParseDirectoryIndex(self, last_known_rev):
[email protected]afe30662011-07-30 01:05:52245 """Parses the Google Storage directory listing into a list of revision
[email protected]eadd95d2012-11-02 22:42:09246 numbers."""
[email protected]afe30662011-07-30 01:05:52247
rob724c9062015-01-22 00:26:42248 def _GetMarkerForRev(revision):
249 if self.is_asan:
250 return '%s-%s/%s-%d.zip' % (
251 self.GetASANPlatformDir(), self.build_type,
252 self.GetASANBaseName(), revision)
253 return '%s%d' % (self._listing_platform_dir, revision)
254
[email protected]afe30662011-07-30 01:05:52255 def _FetchAndParse(url):
256 """Fetches a URL and returns a 2-Tuple of ([revisions], next-marker). If
257 next-marker is not None, then the listing is a partial listing and another
258 fetch should be performed with next-marker being the marker= GET
259 parameter."""
260 handle = urllib.urlopen(url)
261 document = ElementTree.parse(handle)
262
263 # All nodes in the tree are namespaced. Get the root's tag name to extract
264 # the namespace. Etree does namespaces as |{namespace}tag|.
265 root_tag = document.getroot().tag
266 end_ns_pos = root_tag.find('}')
267 if end_ns_pos == -1:
[email protected]4df583c2014-07-31 17:11:55268 raise Exception('Could not locate end namespace for directory index')
[email protected]afe30662011-07-30 01:05:52269 namespace = root_tag[:end_ns_pos + 1]
270
271 # Find the prefix (_listing_platform_dir) and whether or not the list is
272 # truncated.
273 prefix_len = len(document.find(namespace + 'Prefix').text)
274 next_marker = None
275 is_truncated = document.find(namespace + 'IsTruncated')
276 if is_truncated is not None and is_truncated.text.lower() == 'true':
277 next_marker = document.find(namespace + 'NextMarker').text
[email protected]afe30662011-07-30 01:05:52278 # Get a list of all the revisions.
[email protected]afe30662011-07-30 01:05:52279 revisions = []
[email protected]3e7c85322014-06-27 20:27:36280 githash_svn_dict = {}
[email protected]011886692014-08-01 21:00:21281 if self.is_asan:
282 asan_regex = re.compile(r'.*%s-(\d+)\.zip$' % (self.GetASANBaseName()))
283 # Non ASAN builds are in a <revision> directory. The ASAN builds are
284 # flat
285 all_prefixes = document.findall(namespace + 'Contents/' +
286 namespace + 'Key')
287 for prefix in all_prefixes:
288 m = asan_regex.match(prefix.text)
289 if m:
290 try:
291 revisions.append(int(m.group(1)))
292 except ValueError:
293 pass
294 else:
295 all_prefixes = document.findall(namespace + 'CommonPrefixes/' +
296 namespace + 'Prefix')
297 # The <Prefix> nodes have content of the form of
298 # |_listing_platform_dir/revision/|. Strip off the platform dir and the
299 # trailing slash to just have a number.
300 for prefix in all_prefixes:
301 revnum = prefix.text[prefix_len:-1]
302 try:
dimua1dfa0ce2016-03-31 01:08:45303 revnum = int(revnum)
304 revisions.append(revnum)
305 # Notes:
306 # Ignore hash in chromium-browser-snapshots as they are invalid
307 # Resulting in 404 error in fetching pages:
308 # https://chromium.googlesource.com/chromium/src/+/[rev_hash]
[email protected]011886692014-08-01 21:00:21309 except ValueError:
310 pass
[email protected]3e7c85322014-06-27 20:27:36311 return (revisions, next_marker, githash_svn_dict)
[email protected]9639b002013-08-30 14:45:52312
[email protected]afe30662011-07-30 01:05:52313 # Fetch the first list of revisions.
rob724c9062015-01-22 00:26:42314 if last_known_rev:
315 revisions = []
316 # Optimization: Start paging at the last known revision (local cache).
317 next_marker = _GetMarkerForRev(last_known_rev)
318 # Optimization: Stop paging at the last known revision (remote).
319 last_change_rev = GetChromiumRevision(self, self.GetLastChangeURL())
320 if last_known_rev == last_change_rev:
321 return []
322 else:
323 (revisions, next_marker, new_dict) = _FetchAndParse(self.GetListingURL())
324 self.githash_svn_dict.update(new_dict)
325 last_change_rev = None
326
[email protected]afe30662011-07-30 01:05:52327 # If the result list was truncated, refetch with the next marker. Do this
328 # until an entire directory listing is done.
329 while next_marker:
rob724c9062015-01-22 00:26:42330 sys.stdout.write('\rFetching revisions at marker %s' % next_marker)
331 sys.stdout.flush()
332
[email protected]afe30662011-07-30 01:05:52333 next_url = self.GetListingURL(next_marker)
[email protected]3e7c85322014-06-27 20:27:36334 (new_revisions, next_marker, new_dict) = _FetchAndParse(next_url)
[email protected]afe30662011-07-30 01:05:52335 revisions.extend(new_revisions)
[email protected]3e7c85322014-06-27 20:27:36336 self.githash_svn_dict.update(new_dict)
rob724c9062015-01-22 00:26:42337 if last_change_rev and last_change_rev in new_revisions:
338 break
339 sys.stdout.write('\r')
340 sys.stdout.flush()
[email protected]afe30662011-07-30 01:05:52341 return revisions
342
[email protected]6a7a5d62014-07-09 04:45:50343 def _GetSVNRevisionFromGitHashWithoutGitCheckout(self, git_sha1, depot):
[email protected]3e7c85322014-06-27 20:27:36344 json_url = GITHASH_TO_SVN_URL[depot] % git_sha1
[email protected]2e0f2672014-08-13 20:32:58345 response = urllib.urlopen(json_url)
346 if response.getcode() == 200:
347 try:
348 data = json.loads(response.read()[4:])
349 except ValueError:
Raul Tambre57e09d62019-09-22 17:18:52350 print('ValueError for JSON URL: %s' % json_url)
[email protected]2e0f2672014-08-13 20:32:58351 raise ValueError
352 else:
353 raise ValueError
[email protected]3e7c85322014-06-27 20:27:36354 if 'message' in data:
355 message = data['message'].split('\n')
356 message = [line for line in message if line.strip()]
357 search_pattern = re.compile(SEARCH_PATTERN[depot])
358 result = search_pattern.search(message[len(message)-1])
359 if result:
360 return result.group(1)
pshenoyb23a1452014-09-05 22:52:05361 else:
362 if depot == 'chromium':
363 result = re.search(CHROMIUM_SEARCH_PATTERN_OLD,
364 message[len(message)-1])
365 if result:
366 return result.group(1)
Raul Tambre57e09d62019-09-22 17:18:52367 print('Failed to get svn revision number for %s' % git_sha1)
[email protected]1f99f4d2014-07-23 16:44:14368 raise ValueError
[email protected]3e7c85322014-06-27 20:27:36369
[email protected]6a7a5d62014-07-09 04:45:50370 def _GetSVNRevisionFromGitHashFromGitCheckout(self, git_sha1, depot):
371 def _RunGit(command, path):
372 command = ['git'] + command
[email protected]6a7a5d62014-07-09 04:45:50373 shell = sys.platform.startswith('win')
374 proc = subprocess.Popen(command, shell=shell, stdout=subprocess.PIPE,
rob724c9062015-01-22 00:26:42375 stderr=subprocess.PIPE, cwd=path)
[email protected]6a7a5d62014-07-09 04:45:50376 (output, _) = proc.communicate()
[email protected]6a7a5d62014-07-09 04:45:50377 return (output, proc.returncode)
378
rob724c9062015-01-22 00:26:42379 path = self.local_src_path
[email protected]6a7a5d62014-07-09 04:45:50380 if depot == 'blink':
rob724c9062015-01-22 00:26:42381 path = os.path.join(self.local_src_path, 'third_party', 'WebKit')
382 revision = None
383 try:
[email protected]6a7a5d62014-07-09 04:45:50384 command = ['svn', 'find-rev', git_sha1]
385 (git_output, return_code) = _RunGit(command, path)
386 if not return_code:
rob724c9062015-01-22 00:26:42387 revision = git_output.strip('\n')
388 except ValueError:
389 pass
390 if not revision:
391 command = ['log', '-n1', '--format=%s', git_sha1]
392 (git_output, return_code) = _RunGit(command, path)
393 if not return_code:
394 revision = re.match('SVN changes up to revision ([0-9]+)', git_output)
395 revision = revision.group(1) if revision else None
396 if revision:
397 return revision
398 raise ValueError
[email protected]6a7a5d62014-07-09 04:45:50399
400 def GetSVNRevisionFromGitHash(self, git_sha1, depot='chromium'):
rob724c9062015-01-22 00:26:42401 if not self.local_src_path:
[email protected]6a7a5d62014-07-09 04:45:50402 return self._GetSVNRevisionFromGitHashWithoutGitCheckout(git_sha1, depot)
403 else:
404 return self._GetSVNRevisionFromGitHashFromGitCheckout(git_sha1, depot)
405
[email protected]afe30662011-07-30 01:05:52406 def GetRevList(self):
407 """Gets the list of revision numbers between self.good_revision and
408 self.bad_revision."""
rob724c9062015-01-22 00:26:42409
410 cache = {}
411 # The cache is stored in the same directory as bisect-builds.py
412 cache_filename = os.path.join(
413 os.path.abspath(os.path.dirname(__file__)),
414 '.bisect-builds-cache.json')
415 cache_dict_key = self.GetListingURL()
416
417 def _LoadBucketFromCache():
418 if self.use_local_cache:
419 try:
420 with open(cache_filename) as cache_file:
rob1c836052015-05-18 16:34:02421 for (key, value) in json.load(cache_file).items():
422 cache[key] = value
rob724c9062015-01-22 00:26:42423 revisions = cache.get(cache_dict_key, [])
424 githash_svn_dict = cache.get('githash_svn_dict', {})
425 if revisions:
Raul Tambre57e09d62019-09-22 17:18:52426 print('Loaded revisions %d-%d from %s' %
427 (revisions[0], revisions[-1], cache_filename))
rob724c9062015-01-22 00:26:42428 return (revisions, githash_svn_dict)
429 except (EnvironmentError, ValueError):
430 pass
431 return ([], {})
432
433 def _SaveBucketToCache():
434 """Save the list of revisions and the git-svn mappings to a file.
435 The list of revisions is assumed to be sorted."""
436 if self.use_local_cache:
437 cache[cache_dict_key] = revlist_all
438 cache['githash_svn_dict'] = self.githash_svn_dict
439 try:
440 with open(cache_filename, 'w') as cache_file:
441 json.dump(cache, cache_file)
Raul Tambre57e09d62019-09-22 17:18:52442 print('Saved revisions %d-%d to %s' %
443 (revlist_all[0], revlist_all[-1], cache_filename))
rob724c9062015-01-22 00:26:42444 except EnvironmentError:
445 pass
446
[email protected]afe30662011-07-30 01:05:52447 # Download the revlist and filter for just the range between good and bad.
[email protected]eadd95d2012-11-02 22:42:09448 minrev = min(self.good_revision, self.bad_revision)
449 maxrev = max(self.good_revision, self.bad_revision)
rob724c9062015-01-22 00:26:42450
451 (revlist_all, self.githash_svn_dict) = _LoadBucketFromCache()
452 last_known_rev = revlist_all[-1] if revlist_all else 0
453 if last_known_rev < maxrev:
454 revlist_all.extend(map(int, self.ParseDirectoryIndex(last_known_rev)))
455 revlist_all = list(set(revlist_all))
456 revlist_all.sort()
457 _SaveBucketToCache()
[email protected]37ed3172013-09-24 23:49:30458
459 revlist = [x for x in revlist_all if x >= int(minrev) and x <= int(maxrev)]
[email protected]37ed3172013-09-24 23:49:30460
461 # Set good and bad revisions to be legit revisions.
462 if revlist:
463 if self.good_revision < self.bad_revision:
464 self.good_revision = revlist[0]
465 self.bad_revision = revlist[-1]
466 else:
467 self.bad_revision = revlist[0]
468 self.good_revision = revlist[-1]
469
470 # Fix chromium rev so that the deps blink revision matches REVISIONS file.
471 if self.base_url == WEBKIT_BASE_URL:
472 revlist_all.sort()
473 self.good_revision = FixChromiumRevForBlink(revlist,
474 revlist_all,
475 self,
476 self.good_revision)
477 self.bad_revision = FixChromiumRevForBlink(revlist,
478 revlist_all,
479 self,
480 self.bad_revision)
[email protected]afe30662011-07-30 01:05:52481 return revlist
482
prasadv2375e6d2017-03-20 19:23:23483
484def IsMac():
485 return sys.platform.startswith('darwin')
486
487
[email protected]fc3702e2013-11-09 04:23:00488def UnzipFilenameToDir(filename, directory):
489 """Unzip |filename| to |directory|."""
[email protected]afe30662011-07-30 01:05:52490 cwd = os.getcwd()
491 if not os.path.isabs(filename):
492 filename = os.path.join(cwd, filename)
[email protected]bd8dcb92010-03-31 01:05:24493 # Make base.
[email protected]fc3702e2013-11-09 04:23:00494 if not os.path.isdir(directory):
495 os.mkdir(directory)
496 os.chdir(directory)
prasadv2375e6d2017-03-20 19:23:23497
498 # The Python ZipFile does not support symbolic links, which makes it
499 # unsuitable for Mac builds. so use ditto instead.
500 if IsMac():
501 unzip_cmd = ['ditto', '-x', '-k', filename, '.']
502 proc = subprocess.Popen(unzip_cmd, bufsize=0, stdout=subprocess.PIPE,
503 stderr=subprocess.PIPE)
504 proc.communicate()
505 os.chdir(cwd)
506 return
507
508 zf = zipfile.ZipFile(filename)
[email protected]e29c08c2012-09-17 20:50:50509 # Extract files.
510 for info in zf.infolist():
511 name = info.filename
512 if name.endswith('/'): # dir
513 if not os.path.isdir(name):
514 os.makedirs(name)
515 else: # file
[email protected]fc3702e2013-11-09 04:23:00516 directory = os.path.dirname(name)
John Budorick06e5df12015-02-27 17:44:27517 if not os.path.isdir(directory):
[email protected]fc3702e2013-11-09 04:23:00518 os.makedirs(directory)
[email protected]e29c08c2012-09-17 20:50:50519 out = open(name, 'wb')
520 out.write(zf.read(name))
521 out.close()
522 # Set permissions. Permission info in external_attr is shifted 16 bits.
523 os.chmod(name, info.external_attr >> 16L)
524 os.chdir(cwd)
[email protected]bd8dcb92010-03-31 01:05:24525
[email protected]67e0bc62009-09-03 22:06:09526
[email protected]468a9772011-08-09 18:42:00527def FetchRevision(context, rev, filename, quit_event=None, progress_event=None):
[email protected]afe30662011-07-30 01:05:52528 """Downloads and unzips revision |rev|.
529 @param context A PathContext instance.
530 @param rev The Chromium revision number/tag to download.
531 @param filename The destination for the downloaded file.
532 @param quit_event A threading.Event which will be set by the master thread to
533 indicate that the download should be aborted.
[email protected]468a9772011-08-09 18:42:00534 @param progress_event A threading.Event which will be set by the master thread
535 to indicate that the progress of the download should be
536 displayed.
[email protected]afe30662011-07-30 01:05:52537 """
538 def ReportHook(blocknum, blocksize, totalsize):
[email protected]946be752011-10-25 23:34:21539 if quit_event and quit_event.isSet():
[email protected]4df583c2014-07-31 17:11:55540 raise RuntimeError('Aborting download of revision %s' % str(rev))
[email protected]946be752011-10-25 23:34:21541 if progress_event and progress_event.isSet():
[email protected]468a9772011-08-09 18:42:00542 size = blocknum * blocksize
543 if totalsize == -1: # Total size not known.
[email protected]4df583c2014-07-31 17:11:55544 progress = 'Received %d bytes' % size
[email protected]468a9772011-08-09 18:42:00545 else:
546 size = min(totalsize, size)
[email protected]4df583c2014-07-31 17:11:55547 progress = 'Received %d of %d bytes, %.2f%%' % (
[email protected]468a9772011-08-09 18:42:00548 size, totalsize, 100.0 * size / totalsize)
549 # Send a \r to let all progress messages use just one line of output.
[email protected]4df583c2014-07-31 17:11:55550 sys.stdout.write('\r' + progress)
[email protected]468a9772011-08-09 18:42:00551 sys.stdout.flush()
[email protected]afe30662011-07-30 01:05:52552 download_url = context.GetDownloadURL(rev)
553 try:
John Budorick06e5df12015-02-27 17:44:27554 urllib.urlretrieve(download_url, filename, ReportHook)
[email protected]946be752011-10-25 23:34:21555 if progress_event and progress_event.isSet():
Raul Tambre57e09d62019-09-22 17:18:52556 print()
mikecasee2b6ce82015-02-06 18:22:39557
[email protected]4df583c2014-07-31 17:11:55558 except RuntimeError:
[email protected]afe30662011-07-30 01:05:52559 pass
[email protected]7ad66a72009-09-04 17:52:33560
[email protected]7ad66a72009-09-04 17:52:33561
Dominic Mazzoni215e80b2017-11-29 20:05:27562def CopyMissingFileFromCurrentSource(src_glob, dst):
563 """Work around missing files in archives.
564 This happens when archives of Chrome don't contain all of the files
565 needed to build it. In many cases we can work around this using
566 files from the current checkout. The source is in the form of a glob
567 so that it can try to look for possible sources of the file in
568 multiple locations, but we just arbitrarily try the first match.
569
570 Silently fail if this doesn't work because we don't yet have clear
571 markers for builds that require certain files or a way to test
572 whether or not launching Chrome succeeded.
573 """
574 if not os.path.exists(dst):
575 matches = glob.glob(src_glob)
576 if matches:
577 shutil.copy2(matches[0], dst)
578
579
[email protected]4df583c2014-07-31 17:11:55580def RunRevision(context, revision, zip_file, profile, num_runs, command, args):
[email protected]afe30662011-07-30 01:05:52581 """Given a zipped revision, unzip it and run the test."""
Raul Tambre57e09d62019-09-22 17:18:52582 print('Trying revision %s...' % str(revision))
[email protected]3ff00b72011-07-20 21:34:47583
[email protected]afe30662011-07-30 01:05:52584 # Create a temp directory and unzip the revision into it.
[email protected]7ad66a72009-09-04 17:52:33585 cwd = os.getcwd()
586 tempdir = tempfile.mkdtemp(prefix='bisect_tmp')
[email protected]4df583c2014-07-31 17:11:55587 UnzipFilenameToDir(zip_file, tempdir)
dmazzoni76e907d2015-01-22 08:14:49588
Dominic Mazzoni215e80b2017-11-29 20:05:27589 # Hack: Some Chrome OS archives are missing some files; try to copy them
590 # from the local directory.
Dominic Mazzonie84e40b2018-10-08 06:44:45591 if context.platform == 'chromeos' and revision < 591483:
Dominic Mazzoni215e80b2017-11-29 20:05:27592 CopyMissingFileFromCurrentSource('third_party/icu/common/icudtl.dat',
593 '%s/chrome-linux/icudtl.dat' % tempdir)
594 CopyMissingFileFromCurrentSource('*out*/*/libminigbm.so',
595 '%s/chrome-linux/libminigbm.so' % tempdir)
dmazzoni76e907d2015-01-22 08:14:49596
[email protected]7ad66a72009-09-04 17:52:33597 os.chdir(tempdir)
[email protected]67e0bc62009-09-03 22:06:09598
[email protected]5e93cf162012-01-28 02:16:56599 # Run the build as many times as specified.
[email protected]4646a752013-07-19 22:14:34600 testargs = ['--user-data-dir=%s' % profile] + args
[email protected]d0149c5c2012-05-29 21:12:11601 # The sandbox must be run as root on Official Chrome, so bypass it.
Jason Kersey97bb027a2016-05-11 20:10:43602 if (context.flash_path and context.platform.startswith('linux')):
[email protected]d0149c5c2012-05-29 21:12:11603 testargs.append('--no-sandbox')
[email protected]fc3702e2013-11-09 04:23:00604 if context.flash_path:
605 testargs.append('--ppapi-flash-path=%s' % context.flash_path)
606 # We have to pass a large enough Flash version, which currently needs not
607 # be correct. Instead of requiring the user of the script to figure out and
608 # pass the correct version we just spoof it.
609 testargs.append('--ppapi-flash-version=99.9.999.999')
[email protected]d0149c5c2012-05-29 21:12:11610
[email protected]4646a752013-07-19 22:14:34611 runcommand = []
[email protected]61ea90a2013-09-26 10:17:34612 for token in shlex.split(command):
[email protected]4df583c2014-07-31 17:11:55613 if token == '%a':
[email protected]4646a752013-07-19 22:14:34614 runcommand.extend(testargs)
615 else:
[email protected]4df583c2014-07-31 17:11:55616 runcommand.append(
[email protected]011886692014-08-01 21:00:21617 token.replace('%p', os.path.abspath(context.GetLaunchPath(revision))).
618 replace('%s', ' '.join(testargs)))
[email protected]fb61fc32019-04-18 19:47:20619 result = None
[email protected]7ad66a72009-09-04 17:52:33620 try:
[email protected]fb61fc32019-04-18 19:47:20621 for _ in range(num_runs):
622 subproc = subprocess.Popen(
623 runcommand,
624 bufsize=-1,
625 stdout=subprocess.PIPE,
626 stderr=subprocess.PIPE)
627 (stdout, stderr) = subproc.communicate()
628 result = (subproc.returncode, stdout, stderr)
629 if subproc.returncode:
630 break
631 return result
632 finally:
633 os.chdir(cwd)
634 try:
635 shutil.rmtree(tempdir, True)
636 except Exception:
637 pass
[email protected]79f14742010-03-10 01:01:57638
[email protected]cb155a82011-11-29 17:25:34639
Jason Kersey97bb027a2016-05-11 20:10:43640# The arguments status, stdout and stderr are unused.
[email protected]4df583c2014-07-31 17:11:55641# They are present here because this function is passed to Bisect which then
642# calls it with 5 arguments.
643# pylint: disable=W0613
Jason Kersey97bb027a2016-05-11 20:10:43644def AskIsGoodBuild(rev, exit_status, stdout, stderr):
[email protected]4df583c2014-07-31 17:11:55645 """Asks the user whether build |rev| is good or bad."""
Fergal Daly0dd19532019-04-04 07:45:33646 if exit_status:
Raul Tambre57e09d62019-09-22 17:18:52647 print('Chrome exit_status: %d. Use s to see output' % exit_status)
[email protected]79f14742010-03-10 01:01:57648 # Loop until we get a response that we can parse.
[email protected]67e0bc62009-09-03 22:06:09649 while True:
[email protected]4df583c2014-07-31 17:11:55650 response = raw_input('Revision %s is '
wangxianzhud8c4c562015-12-15 23:39:51651 '[(g)ood/(b)ad/(r)etry/(u)nknown/(s)tdout/(q)uit]: ' %
[email protected]53bb6342012-06-01 04:11:00652 str(rev))
wangxianzhud8c4c562015-12-15 23:39:51653 if response in ('g', 'b', 'r', 'u'):
[email protected]53bb6342012-06-01 04:11:00654 return response
wangxianzhud8c4c562015-12-15 23:39:51655 if response == 'q':
[email protected]afe30662011-07-30 01:05:52656 raise SystemExit()
wangxianzhud8c4c562015-12-15 23:39:51657 if response == 's':
Raul Tambre57e09d62019-09-22 17:18:52658 print(stdout)
659 print(stderr)
[email protected]67e0bc62009-09-03 22:06:09660
[email protected]cb155a82011-11-29 17:25:34661
Jason Kersey97bb027a2016-05-11 20:10:43662def IsGoodASANBuild(rev, exit_status, stdout, stderr):
[email protected]011886692014-08-01 21:00:21663 """Determine if an ASAN build |rev| is good or bad
664
665 Will examine stderr looking for the error message emitted by ASAN. If not
666 found then will fallback to asking the user."""
667 if stderr:
668 bad_count = 0
669 for line in stderr.splitlines():
Raul Tambre57e09d62019-09-22 17:18:52670 print(line)
[email protected]011886692014-08-01 21:00:21671 if line.find('ERROR: AddressSanitizer:') != -1:
672 bad_count += 1
673 if bad_count > 0:
Raul Tambre57e09d62019-09-22 17:18:52674 print('Revision %d determined to be bad.' % rev)
[email protected]011886692014-08-01 21:00:21675 return 'b'
Jason Kersey97bb027a2016-05-11 20:10:43676 return AskIsGoodBuild(rev, exit_status, stdout, stderr)
skobes21b5cdfb2016-03-21 23:13:02677
678
Jason Kersey97bb027a2016-05-11 20:10:43679def DidCommandSucceed(rev, exit_status, stdout, stderr):
skobes21b5cdfb2016-03-21 23:13:02680 if exit_status:
Raul Tambre57e09d62019-09-22 17:18:52681 print('Bad revision: %s' % rev)
skobes21b5cdfb2016-03-21 23:13:02682 return 'b'
683 else:
Raul Tambre57e09d62019-09-22 17:18:52684 print('Good revision: %s' % rev)
skobes21b5cdfb2016-03-21 23:13:02685 return 'g'
686
[email protected]011886692014-08-01 21:00:21687
[email protected]53bb6342012-06-01 04:11:00688class DownloadJob(object):
689 """DownloadJob represents a task to download a given Chromium revision."""
[email protected]4df583c2014-07-31 17:11:55690
691 def __init__(self, context, name, rev, zip_file):
[email protected]53bb6342012-06-01 04:11:00692 super(DownloadJob, self).__init__()
693 # Store off the input parameters.
694 self.context = context
695 self.name = name
696 self.rev = rev
[email protected]4df583c2014-07-31 17:11:55697 self.zip_file = zip_file
[email protected]53bb6342012-06-01 04:11:00698 self.quit_event = threading.Event()
699 self.progress_event = threading.Event()
[email protected]4df583c2014-07-31 17:11:55700 self.thread = None
[email protected]53bb6342012-06-01 04:11:00701
702 def Start(self):
703 """Starts the download."""
704 fetchargs = (self.context,
705 self.rev,
[email protected]4df583c2014-07-31 17:11:55706 self.zip_file,
[email protected]53bb6342012-06-01 04:11:00707 self.quit_event,
708 self.progress_event)
709 self.thread = threading.Thread(target=FetchRevision,
710 name=self.name,
711 args=fetchargs)
712 self.thread.start()
713
714 def Stop(self):
715 """Stops the download which must have been started previously."""
[email protected]4df583c2014-07-31 17:11:55716 assert self.thread, 'DownloadJob must be started before Stop is called.'
[email protected]53bb6342012-06-01 04:11:00717 self.quit_event.set()
718 self.thread.join()
[email protected]4df583c2014-07-31 17:11:55719 os.unlink(self.zip_file)
[email protected]53bb6342012-06-01 04:11:00720
721 def WaitFor(self):
722 """Prints a message and waits for the download to complete. The download
723 must have been started previously."""
[email protected]4df583c2014-07-31 17:11:55724 assert self.thread, 'DownloadJob must be started before WaitFor is called.'
Raul Tambre57e09d62019-09-22 17:18:52725 print('Downloading revision %s...' % str(self.rev))
[email protected]53bb6342012-06-01 04:11:00726 self.progress_event.set() # Display progress of download.
rob8a4543f2016-01-20 00:43:59727 try:
728 while self.thread.isAlive():
729 # The parameter to join is needed to keep the main thread responsive to
730 # signals. Without it, the program will not respond to interruptions.
731 self.thread.join(1)
732 except (KeyboardInterrupt, SystemExit):
733 self.Stop()
734 raise
[email protected]53bb6342012-06-01 04:11:00735
736
skobes21b5cdfb2016-03-21 23:13:02737def VerifyEndpoint(fetch, context, rev, profile, num_runs, command, try_args,
738 evaluate, expected_answer):
739 fetch.WaitFor()
740 try:
Roman Sorokin760f06cd2019-12-24 08:35:41741 answer = 'r'
742 # This is intended to allow evaluate() to return 'r' to retry RunRevision.
743 while answer == 'r':
744 (exit_status, stdout, stderr) = RunRevision(
745 context, rev, fetch.zip_file, profile, num_runs, command, try_args)
746 answer = evaluate(rev, exit_status, stdout, stderr);
skobes21b5cdfb2016-03-21 23:13:02747 except Exception, e:
Raul Tambre57e09d62019-09-22 17:18:52748 print(e, file=sys.stderr)
Lei Zhang2fa76302018-11-09 20:16:31749 raise SystemExit
Roman Sorokin760f06cd2019-12-24 08:35:41750 if (answer != expected_answer):
Raul Tambre57e09d62019-09-22 17:18:52751 print('Unexpected result at a range boundary! Your range is not correct.')
skobes21b5cdfb2016-03-21 23:13:02752 raise SystemExit
753
754
[email protected]2e0f2672014-08-13 20:32:58755def Bisect(context,
[email protected]5e93cf162012-01-28 02:16:56756 num_runs=1,
[email protected]4df583c2014-07-31 17:11:55757 command='%p %a',
[email protected]60ac66e32011-07-18 16:08:25758 try_args=(),
[email protected]afe30662011-07-30 01:05:52759 profile=None,
skobes21b5cdfb2016-03-21 23:13:02760 evaluate=AskIsGoodBuild,
761 verify_range=False):
[email protected]afe30662011-07-30 01:05:52762 """Given known good and known bad revisions, run a binary search on all
763 archived revisions to determine the last known good revision.
[email protected]60ac66e32011-07-18 16:08:25764
[email protected]2e0f2672014-08-13 20:32:58765 @param context PathContext object initialized with user provided parameters.
[email protected]5e93cf162012-01-28 02:16:56766 @param num_runs Number of times to run each build for asking good/bad.
[email protected]afe30662011-07-30 01:05:52767 @param try_args A tuple of arguments to pass to the test application.
768 @param profile The name of the user profile to run with.
[email protected]53bb6342012-06-01 04:11:00769 @param evaluate A function which returns 'g' if the argument build is good,
770 'b' if it's bad or 'u' if unknown.
skobes21b5cdfb2016-03-21 23:13:02771 @param verify_range If true, tests the first and last revisions in the range
772 before proceeding with the bisect.
[email protected]afe30662011-07-30 01:05:52773
774 Threading is used to fetch Chromium revisions in the background, speeding up
775 the user's experience. For example, suppose the bounds of the search are
776 good_rev=0, bad_rev=100. The first revision to be checked is 50. Depending on
777 whether revision 50 is good or bad, the next revision to check will be either
778 25 or 75. So, while revision 50 is being checked, the script will download
779 revisions 25 and 75 in the background. Once the good/bad verdict on rev 50 is
780 known:
781
782 - If rev 50 is good, the download of rev 25 is cancelled, and the next test
783 is run on rev 75.
784
785 - If rev 50 is bad, the download of rev 75 is cancelled, and the next test
786 is run on rev 25.
[email protected]60ac66e32011-07-18 16:08:25787 """
788
[email protected]afe30662011-07-30 01:05:52789 if not profile:
790 profile = 'profile'
791
[email protected]2e0f2672014-08-13 20:32:58792 good_rev = context.good_revision
793 bad_rev = context.bad_revision
[email protected]afe30662011-07-30 01:05:52794 cwd = os.getcwd()
795
Raul Tambre57e09d62019-09-22 17:18:52796 print('Downloading list of known revisions...', end=' ')
Jason Kersey97bb027a2016-05-11 20:10:43797 if not context.use_local_cache:
Raul Tambre57e09d62019-09-22 17:18:52798 print('(use --use-local-cache to cache and re-use the list of revisions)')
[email protected]28a3c122014-08-09 11:04:51799 else:
Raul Tambre57e09d62019-09-22 17:18:52800 print()
[email protected]d0149c5c2012-05-29 21:12:11801 _GetDownloadPath = lambda rev: os.path.join(cwd,
802 '%s-%s' % (str(rev), context.archive_name))
Jason Kersey97bb027a2016-05-11 20:10:43803 revlist = context.GetRevList()
[email protected]afe30662011-07-30 01:05:52804
805 # Get a list of revisions to bisect across.
806 if len(revlist) < 2: # Don't have enough builds to bisect.
807 msg = 'We don\'t have enough builds to bisect. revlist: %s' % revlist
808 raise RuntimeError(msg)
809
810 # Figure out our bookends and first pivot point; fetch the pivot revision.
[email protected]eadd95d2012-11-02 22:42:09811 minrev = 0
812 maxrev = len(revlist) - 1
813 pivot = maxrev / 2
[email protected]afe30662011-07-30 01:05:52814 rev = revlist[pivot]
skobes21b5cdfb2016-03-21 23:13:02815 fetch = DownloadJob(context, 'initial_fetch', rev, _GetDownloadPath(rev))
[email protected]eadd95d2012-11-02 22:42:09816 fetch.Start()
skobes21b5cdfb2016-03-21 23:13:02817
818 if verify_range:
819 minrev_fetch = DownloadJob(
820 context, 'minrev_fetch', revlist[minrev],
821 _GetDownloadPath(revlist[minrev]))
822 maxrev_fetch = DownloadJob(
823 context, 'maxrev_fetch', revlist[maxrev],
824 _GetDownloadPath(revlist[maxrev]))
825 minrev_fetch.Start()
826 maxrev_fetch.Start()
827 try:
828 VerifyEndpoint(minrev_fetch, context, revlist[minrev], profile, num_runs,
829 command, try_args, evaluate, 'b' if bad_rev < good_rev else 'g')
830 VerifyEndpoint(maxrev_fetch, context, revlist[maxrev], profile, num_runs,
831 command, try_args, evaluate, 'g' if bad_rev < good_rev else 'b')
832 except (KeyboardInterrupt, SystemExit):
Raul Tambre57e09d62019-09-22 17:18:52833 print('Cleaning up...')
skobes21b5cdfb2016-03-21 23:13:02834 fetch.Stop()
835 sys.exit(0)
836 finally:
837 minrev_fetch.Stop()
838 maxrev_fetch.Stop()
839
[email protected]eadd95d2012-11-02 22:42:09840 fetch.WaitFor()
[email protected]60ac66e32011-07-18 16:08:25841
842 # Binary search time!
Bruce Dawson62257412020-01-17 17:39:53843 prefetch_revisions = True
[email protected]4df583c2014-07-31 17:11:55844 while fetch and fetch.zip_file and maxrev - minrev > 1:
[email protected]eadd95d2012-11-02 22:42:09845 if bad_rev < good_rev:
[email protected]4df583c2014-07-31 17:11:55846 min_str, max_str = 'bad', 'good'
[email protected]eadd95d2012-11-02 22:42:09847 else:
[email protected]4df583c2014-07-31 17:11:55848 min_str, max_str = 'good', 'bad'
Raul Tambre57e09d62019-09-22 17:18:52849 print(
850 'Bisecting range [%s (%s), %s (%s)], '
851 'roughly %d steps left.' % (revlist[minrev], min_str, revlist[maxrev],
852 max_str, int(maxrev - minrev).bit_length()))
[email protected]eadd95d2012-11-02 22:42:09853
[email protected]afe30662011-07-30 01:05:52854 # Pre-fetch next two possible pivots
855 # - down_pivot is the next revision to check if the current revision turns
856 # out to be bad.
857 # - up_pivot is the next revision to check if the current revision turns
858 # out to be good.
[email protected]eadd95d2012-11-02 22:42:09859 down_pivot = int((pivot - minrev) / 2) + minrev
Bruce Dawson62257412020-01-17 17:39:53860 if prefetch_revisions:
861 down_fetch = None
862 if down_pivot != pivot and down_pivot != minrev:
863 down_rev = revlist[down_pivot]
864 down_fetch = DownloadJob(context, 'down_fetch', down_rev,
865 _GetDownloadPath(down_rev))
866 down_fetch.Start()
[email protected]60ac66e32011-07-18 16:08:25867
[email protected]eadd95d2012-11-02 22:42:09868 up_pivot = int((maxrev - pivot) / 2) + pivot
Bruce Dawson62257412020-01-17 17:39:53869 if prefetch_revisions:
870 up_fetch = None
871 if up_pivot != pivot and up_pivot != maxrev:
872 up_rev = revlist[up_pivot]
873 up_fetch = DownloadJob(context, 'up_fetch', up_rev,
874 _GetDownloadPath(up_rev))
875 up_fetch.Start()
[email protected]60ac66e32011-07-18 16:08:25876
[email protected]afe30662011-07-30 01:05:52877 # Run test on the pivot revision.
skobes21b5cdfb2016-03-21 23:13:02878 exit_status = None
[email protected]e29c08c2012-09-17 20:50:50879 stdout = None
880 stderr = None
881 try:
skobes21b5cdfb2016-03-21 23:13:02882 (exit_status, stdout, stderr) = RunRevision(
883 context, rev, fetch.zip_file, profile, num_runs, command, try_args)
[email protected]e29c08c2012-09-17 20:50:50884 except Exception, e:
Raul Tambre57e09d62019-09-22 17:18:52885 print(e, file=sys.stderr)
[email protected]60ac66e32011-07-18 16:08:25886
[email protected]53bb6342012-06-01 04:11:00887 # Call the evaluate function to see if the current revision is good or bad.
[email protected]afe30662011-07-30 01:05:52888 # On that basis, kill one of the background downloads and complete the
889 # other, as described in the comments above.
890 try:
Jason Kersey97bb027a2016-05-11 20:10:43891 answer = evaluate(rev, exit_status, stdout, stderr)
Bruce Dawson62257412020-01-17 17:39:53892 prefetch_revisions = True
[email protected]4df583c2014-07-31 17:11:55893 if ((answer == 'g' and good_rev < bad_rev)
894 or (answer == 'b' and bad_rev < good_rev)):
[email protected]1d4a06242013-08-20 22:53:12895 fetch.Stop()
[email protected]eadd95d2012-11-02 22:42:09896 minrev = pivot
[email protected]53bb6342012-06-01 04:11:00897 if down_fetch:
898 down_fetch.Stop() # Kill the download of the older revision.
[email protected]1d4a06242013-08-20 22:53:12899 fetch = None
[email protected]53bb6342012-06-01 04:11:00900 if up_fetch:
901 up_fetch.WaitFor()
[email protected]afe30662011-07-30 01:05:52902 pivot = up_pivot
[email protected]eadd95d2012-11-02 22:42:09903 fetch = up_fetch
[email protected]4df583c2014-07-31 17:11:55904 elif ((answer == 'b' and good_rev < bad_rev)
905 or (answer == 'g' and bad_rev < good_rev)):
[email protected]1d4a06242013-08-20 22:53:12906 fetch.Stop()
[email protected]eadd95d2012-11-02 22:42:09907 maxrev = pivot
[email protected]53bb6342012-06-01 04:11:00908 if up_fetch:
909 up_fetch.Stop() # Kill the download of the newer revision.
[email protected]1d4a06242013-08-20 22:53:12910 fetch = None
[email protected]53bb6342012-06-01 04:11:00911 if down_fetch:
912 down_fetch.WaitFor()
[email protected]afe30662011-07-30 01:05:52913 pivot = down_pivot
[email protected]eadd95d2012-11-02 22:42:09914 fetch = down_fetch
[email protected]1d4a06242013-08-20 22:53:12915 elif answer == 'r':
Bruce Dawson62257412020-01-17 17:39:53916 # Don't redundantly prefetch.
917 prefetch_revisions = False
[email protected]53bb6342012-06-01 04:11:00918 elif answer == 'u':
919 # Nuke the revision from the revlist and choose a new pivot.
[email protected]1d4a06242013-08-20 22:53:12920 fetch.Stop()
[email protected]53bb6342012-06-01 04:11:00921 revlist.pop(pivot)
[email protected]eadd95d2012-11-02 22:42:09922 maxrev -= 1 # Assumes maxrev >= pivot.
[email protected]53bb6342012-06-01 04:11:00923
[email protected]eadd95d2012-11-02 22:42:09924 if maxrev - minrev > 1:
[email protected]53bb6342012-06-01 04:11:00925 # Alternate between using down_pivot or up_pivot for the new pivot
926 # point, without affecting the range. Do this instead of setting the
927 # pivot to the midpoint of the new range because adjacent revisions
928 # are likely affected by the same issue that caused the (u)nknown
929 # response.
930 if up_fetch and down_fetch:
931 fetch = [up_fetch, down_fetch][len(revlist) % 2]
932 elif up_fetch:
933 fetch = up_fetch
934 else:
935 fetch = down_fetch
936 fetch.WaitFor()
937 if fetch == up_fetch:
938 pivot = up_pivot - 1 # Subtracts 1 because revlist was resized.
939 else:
940 pivot = down_pivot
[email protected]53bb6342012-06-01 04:11:00941
942 if down_fetch and fetch != down_fetch:
943 down_fetch.Stop()
944 if up_fetch and fetch != up_fetch:
945 up_fetch.Stop()
946 else:
[email protected]4df583c2014-07-31 17:11:55947 assert False, 'Unexpected return value from evaluate(): ' + answer
skobes21b5cdfb2016-03-21 23:13:02948 except (KeyboardInterrupt, SystemExit):
Raul Tambre57e09d62019-09-22 17:18:52949 print('Cleaning up...')
skobes21b5cdfb2016-03-21 23:13:02950 for f in [_GetDownloadPath(rev),
951 _GetDownloadPath(revlist[down_pivot]),
[email protected]5e93cf162012-01-28 02:16:56952 _GetDownloadPath(revlist[up_pivot])]:
[email protected]afe30662011-07-30 01:05:52953 try:
954 os.unlink(f)
955 except OSError:
956 pass
957 sys.exit(0)
958
959 rev = revlist[pivot]
960
[email protected]2e0f2672014-08-13 20:32:58961 return (revlist[minrev], revlist[maxrev], context)
[email protected]60ac66e32011-07-18 16:08:25962
963
pshenoycd6bd682014-09-10 20:50:22964def GetBlinkDEPSRevisionForChromiumRevision(self, rev):
[email protected]4c6fec6b2013-09-17 17:44:08965 """Returns the blink revision that was in REVISIONS file at
[email protected]b2fe7f22011-10-25 22:58:31966 chromium revision |rev|."""
pshenoycd6bd682014-09-10 20:50:22967
968 def _GetBlinkRev(url, blink_re):
969 m = blink_re.search(url.read())
970 url.close()
971 if m:
fmalitaa898d222016-07-12 22:29:03972 return m.group(1)
pshenoycd6bd682014-09-10 20:50:22973
Di Mu08c59682016-07-11 23:05:07974 url = urllib.urlopen(DEPS_FILE % GetGitHashFromSVNRevision(rev))
pshenoycd6bd682014-09-10 20:50:22975 if url.getcode() == 200:
Di Mu08c59682016-07-11 23:05:07976 blink_re = re.compile(r'webkit_revision\D*\d+;\D*\d+;(\w+)')
977 blink_git_sha = _GetBlinkRev(url, blink_re)
978 return self.GetSVNRevisionFromGitHash(blink_git_sha, 'blink')
pshenoycd6bd682014-09-10 20:50:22979 raise Exception('Could not get Blink revision for Chromium rev %d' % rev)
[email protected]37ed3172013-09-24 23:49:30980
981
[email protected]2e0f2672014-08-13 20:32:58982def GetBlinkRevisionForChromiumRevision(context, rev):
[email protected]37ed3172013-09-24 23:49:30983 """Returns the blink revision that was in REVISIONS file at
984 chromium revision |rev|."""
[email protected]3e7c85322014-06-27 20:27:36985 def _IsRevisionNumber(revision):
986 if isinstance(revision, int):
987 return True
988 else:
989 return revision.isdigit()
[email protected]2e0f2672014-08-13 20:32:58990 if str(rev) in context.githash_svn_dict:
991 rev = context.githash_svn_dict[str(rev)]
992 file_url = '%s/%s%s/REVISIONS' % (context.base_url,
993 context._listing_platform_dir, rev)
[email protected]4c6fec6b2013-09-17 17:44:08994 url = urllib.urlopen(file_url)
[email protected]2e0f2672014-08-13 20:32:58995 if url.getcode() == 200:
996 try:
997 data = json.loads(url.read())
998 except ValueError:
Raul Tambre57e09d62019-09-22 17:18:52999 print('ValueError for JSON URL: %s' % file_url)
[email protected]2e0f2672014-08-13 20:32:581000 raise ValueError
1001 else:
1002 raise ValueError
[email protected]b2fe7f22011-10-25 22:58:311003 url.close()
[email protected]4c6fec6b2013-09-17 17:44:081004 if 'webkit_revision' in data:
[email protected]3e7c85322014-06-27 20:27:361005 blink_rev = data['webkit_revision']
1006 if not _IsRevisionNumber(blink_rev):
[email protected]2e0f2672014-08-13 20:32:581007 blink_rev = int(context.GetSVNRevisionFromGitHash(blink_rev, 'blink'))
[email protected]3e7c85322014-06-27 20:27:361008 return blink_rev
[email protected]b2fe7f22011-10-25 22:58:311009 else:
[email protected]ff50d1c2013-04-17 18:49:361010 raise Exception('Could not get blink revision for cr rev %d' % rev)
[email protected]b2fe7f22011-10-25 22:58:311011
[email protected]4df583c2014-07-31 17:11:551012
[email protected]37ed3172013-09-24 23:49:301013def FixChromiumRevForBlink(revisions_final, revisions, self, rev):
1014 """Returns the chromium revision that has the correct blink revision
1015 for blink bisect, DEPS and REVISIONS file might not match since
1016 blink snapshots point to tip of tree blink.
1017 Note: The revisions_final variable might get modified to include
1018 additional revisions."""
pshenoycd6bd682014-09-10 20:50:221019 blink_deps_rev = GetBlinkDEPSRevisionForChromiumRevision(self, rev)
[email protected]37ed3172013-09-24 23:49:301020
1021 while (GetBlinkRevisionForChromiumRevision(self, rev) > blink_deps_rev):
1022 idx = revisions.index(rev)
1023 if idx > 0:
1024 rev = revisions[idx-1]
1025 if rev not in revisions_final:
1026 revisions_final.insert(0, rev)
1027
1028 revisions_final.sort()
1029 return rev
[email protected]b2fe7f22011-10-25 22:58:311030
[email protected]4df583c2014-07-31 17:11:551031
[email protected]5980b752014-07-02 00:34:401032def GetChromiumRevision(context, url):
[email protected]801fb652012-07-20 20:13:501033 """Returns the chromium revision read from given URL."""
1034 try:
1035 # Location of the latest build revision number
[email protected]5980b752014-07-02 00:34:401036 latest_revision = urllib.urlopen(url).read()
1037 if latest_revision.isdigit():
1038 return int(latest_revision)
1039 return context.GetSVNRevisionFromGitHash(latest_revision)
[email protected]4df583c2014-07-31 17:11:551040 except Exception:
Raul Tambre57e09d62019-09-22 17:18:521041 print('Could not determine latest revision. This could be bad...')
[email protected]801fb652012-07-20 20:13:501042 return 999999999
1043
pshenoycd6bd682014-09-10 20:50:221044def GetGitHashFromSVNRevision(svn_revision):
1045 crrev_url = CRREV_URL + str(svn_revision)
1046 url = urllib.urlopen(crrev_url)
1047 if url.getcode() == 200:
1048 data = json.loads(url.read())
1049 if 'git_sha' in data:
1050 return data['git_sha']
1051
pshenoy9ce271f2014-09-02 22:14:051052def PrintChangeLog(min_chromium_rev, max_chromium_rev):
1053 """Prints the changelog URL."""
1054
Raul Tambre57e09d62019-09-22 17:18:521055 print(' ' + CHANGELOG_URL % (GetGitHashFromSVNRevision(min_chromium_rev),
1056 GetGitHashFromSVNRevision(max_chromium_rev)))
1057
pshenoy9ce271f2014-09-02 22:14:051058
elawrence446bcc32017-04-14 17:18:511059def error_internal_option(option, opt, value, parser):
[email protected]fb61fc32019-04-18 19:47:201060 raise optparse.OptionValueError(
1061 'The -o and -r options are only\navailable in the internal version of '
1062 'this script. Google\nemployees should visit http://go/bisect-builds '
1063 'for\nconfiguration instructions.')
[email protected]801fb652012-07-20 20:13:501064
[email protected]67e0bc62009-09-03 22:06:091065def main():
[email protected]2c1d2732009-10-29 19:52:171066 usage = ('%prog [options] [-- chromium-options]\n'
[email protected]887c9182013-02-12 20:30:311067 'Perform binary search on the snapshot builds to find a minimal\n'
1068 'range of revisions where a behavior change happened. The\n'
1069 'behaviors are described as "good" and "bad".\n'
1070 'It is NOT assumed that the behavior of the later revision is\n'
[email protected]09c58da2013-01-07 21:30:171071 'the bad one.\n'
[email protected]178aab72010-10-08 17:21:381072 '\n'
[email protected]887c9182013-02-12 20:30:311073 'Revision numbers should use\n'
[email protected]887c9182013-02-12 20:30:311074 ' SVN revisions (e.g. 123456) for chromium builds, from trunk.\n'
1075 ' Use base_trunk_revision from http://omahaproxy.appspot.com/\n'
1076 ' for earlier revs.\n'
1077 ' Chrome\'s about: build number and omahaproxy branch_revision\n'
1078 ' are incorrect, they are from branches.\n'
1079 '\n'
Bruce Dawsone3573052020-06-29 23:14:351080 'Use "-- <args-to-pass-to-chromium>" to pass arbitrary extra \n'
1081 'arguments to the test binaries.\n'
1082 'E.g., add "-- --no-first-run" to bypass the first run prompts.')
[email protected]7ad66a72009-09-04 17:52:331083 parser = optparse.OptionParser(usage=usage)
[email protected]1a45d222009-09-19 01:58:571084 # Strangely, the default help output doesn't include the choice list.
mikecasea8cd284c2014-12-02 21:30:581085 choices = ['mac', 'mac64', 'win', 'win64', 'linux', 'linux64', 'linux-arm',
dmazzoni76e907d2015-01-22 08:14:491086 'chromeos']
[email protected]7ad66a72009-09-04 17:52:331087 parser.add_option('-a', '--archive',
[email protected]4df583c2014-07-31 17:11:551088 choices=choices,
1089 help='The buildbot archive to bisect [%s].' %
1090 '|'.join(choices))
[email protected]4df583c2014-07-31 17:11:551091 parser.add_option('-b', '--bad',
1092 type='str',
1093 help='A bad revision to start bisection. '
1094 'May be earlier or later than the good revision. '
1095 'Default is HEAD.')
1096 parser.add_option('-f', '--flash_path',
1097 type='str',
1098 help='Absolute path to a recent Adobe Pepper Flash '
1099 'binary to be used in this bisection (e.g. '
1100 'on Windows C:\...\pepflashplayer.dll and on Linux '
1101 '/opt/google/chrome/PepperFlash/'
1102 'libpepflashplayer.so).')
[email protected]4df583c2014-07-31 17:11:551103 parser.add_option('-g', '--good',
1104 type='str',
1105 help='A good revision to start bisection. ' +
1106 'May be earlier or later than the bad revision. ' +
1107 'Default is 0.')
1108 parser.add_option('-p', '--profile', '--user-data-dir',
1109 type='str',
1110 default='profile',
1111 help='Profile to use; this will not reset every run. '
1112 'Defaults to a clean profile.')
1113 parser.add_option('-t', '--times',
1114 type='int',
1115 default=1,
1116 help='Number of times to run each build before asking '
1117 'if it\'s good or bad. Temporary profiles are reused.')
Bruce Dawsone3573052020-06-29 23:14:351118 parser.add_option('-c',
1119 '--command',
[email protected]4df583c2014-07-31 17:11:551120 type='str',
1121 default='%p %a',
1122 help='Command to execute. %p and %a refer to Chrome '
Bruce Dawsone3573052020-06-29 23:14:351123 'executable and specified extra arguments respectively. '
1124 'Use %s to specify all extra arguments as one string. '
1125 'Defaults to "%p %a". Note that any extra paths specified '
1126 'should be absolute. If you just need to append an '
1127 'argument to the Chrome command line use "-- '
1128 '<args-to-pass-to-chromium>" instead.')
[email protected]4df583c2014-07-31 17:11:551129 parser.add_option('-l', '--blink',
1130 action='store_true',
1131 help='Use Blink bisect instead of Chromium. ')
1132 parser.add_option('', '--not-interactive',
1133 action='store_true',
1134 default=False,
1135 help='Use command exit code to tell good/bad revision.')
[email protected]011886692014-08-01 21:00:211136 parser.add_option('--asan',
1137 dest='asan',
1138 action='store_true',
1139 default=False,
1140 help='Allow the script to bisect ASAN builds')
rob724c9062015-01-22 00:26:421141 parser.add_option('--use-local-cache',
1142 dest='use_local_cache',
[email protected]6a7a5d62014-07-09 04:45:501143 action='store_true',
1144 default=False,
rob724c9062015-01-22 00:26:421145 help='Use a local file in the current directory to cache '
1146 'a list of known revisions to speed up the '
1147 'initialization of this script.')
skobes21b5cdfb2016-03-21 23:13:021148 parser.add_option('--verify-range',
1149 dest='verify_range',
1150 action='store_true',
1151 default=False,
1152 help='Test the first and last revisions in the range ' +
1153 'before proceeding with the bisect.')
elawrence446bcc32017-04-14 17:18:511154 parser.add_option("-r", action="callback", callback=error_internal_option)
1155 parser.add_option("-o", action="callback", callback=error_internal_option)
[email protected]b3b20512013-08-26 18:51:041156
[email protected]7ad66a72009-09-04 17:52:331157 (opts, args) = parser.parse_args()
1158
1159 if opts.archive is None:
Raul Tambre57e09d62019-09-22 17:18:521160 print('Error: missing required parameter: --archive')
1161 print()
[email protected]7ad66a72009-09-04 17:52:331162 parser.print_help()
1163 return 1
1164
[email protected]011886692014-08-01 21:00:211165 if opts.asan:
1166 supported_platforms = ['linux', 'mac', 'win']
1167 if opts.archive not in supported_platforms:
Raul Tambre57e09d62019-09-22 17:18:521168 print('Error: ASAN bisecting only supported on these platforms: [%s].' %
1169 ('|'.join(supported_platforms)))
[email protected]011886692014-08-01 21:00:211170 return 1
[email protected]011886692014-08-01 21:00:211171
1172 if opts.asan:
1173 base_url = ASAN_BASE_URL
1174 elif opts.blink:
[email protected]4c6fec6b2013-09-17 17:44:081175 base_url = WEBKIT_BASE_URL
1176 else:
1177 base_url = CHROMIUM_BASE_URL
1178
[email protected]183706d92011-06-10 13:06:221179 # Create the context. Initialize 0 for the revisions as they are set below.
[email protected]2e0f2672014-08-13 20:32:581180 context = PathContext(base_url, opts.archive, opts.good, opts.bad,
Jason Kersey97bb027a2016-05-11 20:10:431181 opts.asan, opts.use_local_cache,
vitalybuka4d1e1e412015-07-06 17:21:061182 opts.flash_path)
mikecasea8cd284c2014-12-02 21:30:581183
[email protected]67e0bc62009-09-03 22:06:091184 # Pick a starting point, try to get HEAD for this.
[email protected]2e0f2672014-08-13 20:32:581185 if not opts.bad:
1186 context.bad_revision = '999.0.0.0'
1187 context.bad_revision = GetChromiumRevision(
1188 context, context.GetLastChangeURL())
[email protected]67e0bc62009-09-03 22:06:091189
1190 # Find out when we were good.
[email protected]2e0f2672014-08-13 20:32:581191 if not opts.good:
Jason Kersey97bb027a2016-05-11 20:10:431192 context.good_revision = 0
[email protected]801fb652012-07-20 20:13:501193
[email protected]fc3702e2013-11-09 04:23:001194 if opts.flash_path:
[email protected]2e0f2672014-08-13 20:32:581195 msg = 'Could not find Flash binary at %s' % opts.flash_path
1196 assert os.path.exists(opts.flash_path), msg
[email protected]fc3702e2013-11-09 04:23:001197
Jason Kersey97bb027a2016-05-11 20:10:431198 context.good_revision = int(context.good_revision)
1199 context.bad_revision = int(context.bad_revision)
[email protected]801fb652012-07-20 20:13:501200
[email protected]5e93cf162012-01-28 02:16:561201 if opts.times < 1:
1202 print('Number of times to run (%d) must be greater than or equal to 1.' %
1203 opts.times)
1204 parser.print_help()
1205 return 1
1206
skobes21b5cdfb2016-03-21 23:13:021207 if opts.not_interactive:
1208 evaluator = DidCommandSucceed
1209 elif opts.asan:
[email protected]011886692014-08-01 21:00:211210 evaluator = IsGoodASANBuild
1211 else:
1212 evaluator = AskIsGoodBuild
1213
[email protected]2e0f2672014-08-13 20:32:581214 # Save these revision numbers to compare when showing the changelog URL
1215 # after the bisect.
1216 good_rev = context.good_revision
1217 bad_rev = context.bad_revision
1218
1219 (min_chromium_rev, max_chromium_rev, context) = Bisect(
1220 context, opts.times, opts.command, args, opts.profile,
skobes21b5cdfb2016-03-21 23:13:021221 evaluator, opts.verify_range)
[email protected]67e0bc62009-09-03 22:06:091222
[email protected]ff50d1c2013-04-17 18:49:361223 # Get corresponding blink revisions.
[email protected]b2fe7f22011-10-25 22:58:311224 try:
[email protected]4c6fec6b2013-09-17 17:44:081225 min_blink_rev = GetBlinkRevisionForChromiumRevision(context,
1226 min_chromium_rev)
1227 max_blink_rev = GetBlinkRevisionForChromiumRevision(context,
1228 max_chromium_rev)
[email protected]4df583c2014-07-31 17:11:551229 except Exception:
[email protected]b2fe7f22011-10-25 22:58:311230 # Silently ignore the failure.
[email protected]ff50d1c2013-04-17 18:49:361231 min_blink_rev, max_blink_rev = 0, 0
[email protected]b2fe7f22011-10-25 22:58:311232
[email protected]3bdaa4752013-09-30 20:13:361233 if opts.blink:
1234 # We're done. Let the user know the results in an official manner.
1235 if good_rev > bad_rev:
Raul Tambre57e09d62019-09-22 17:18:521236 print(DONE_MESSAGE_GOOD_MAX % (str(min_blink_rev), str(max_blink_rev)))
[email protected]3bdaa4752013-09-30 20:13:361237 else:
Raul Tambre57e09d62019-09-22 17:18:521238 print(DONE_MESSAGE_GOOD_MIN % (str(min_blink_rev), str(max_blink_rev)))
[email protected]eadd95d2012-11-02 22:42:091239
Raul Tambre57e09d62019-09-22 17:18:521240 print('BLINK CHANGELOG URL:')
1241 print(' ' + BLINK_CHANGELOG_URL % (max_blink_rev, min_blink_rev))
[email protected]3bdaa4752013-09-30 20:13:361242
[email protected]d0149c5c2012-05-29 21:12:111243 else:
[email protected]3bdaa4752013-09-30 20:13:361244 # We're done. Let the user know the results in an official manner.
1245 if good_rev > bad_rev:
Raul Tambre57e09d62019-09-22 17:18:521246 print(DONE_MESSAGE_GOOD_MAX % (str(min_chromium_rev),
1247 str(max_chromium_rev)))
[email protected]3bdaa4752013-09-30 20:13:361248 else:
Raul Tambre57e09d62019-09-22 17:18:521249 print(DONE_MESSAGE_GOOD_MIN % (str(min_chromium_rev),
1250 str(max_chromium_rev)))
[email protected]3bdaa4752013-09-30 20:13:361251 if min_blink_rev != max_blink_rev:
[email protected]4df583c2014-07-31 17:11:551252 print ('NOTE: There is a Blink roll in the range, '
1253 'you might also want to do a Blink bisect.')
[email protected]3bdaa4752013-09-30 20:13:361254
Raul Tambre57e09d62019-09-22 17:18:521255 print('CHANGELOG URL:')
Jason Kersey97bb027a2016-05-11 20:10:431256 PrintChangeLog(min_chromium_rev, max_chromium_rev)
[email protected]cb155a82011-11-29 17:25:341257
[email protected]4df583c2014-07-31 17:11:551258
[email protected]67e0bc62009-09-03 22:06:091259if __name__ == '__main__':
[email protected]7ad66a72009-09-04 17:52:331260 sys.exit(main())