Clarify output from bisect-builds so that it's more clear what the

tools/bisect-builds.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Snapshot Build Bisect Tool
 
 This script bisects a snapshot archive using binary search. It starts at
 a bad revision (it will try to guess HEAD) and asks for a last known-good
 revision. It will then binary search across this revision range by downloading,
@@ skipping 11 matching lines @@
 CHANGELOG_URL = 'http://build.chromium.org/f/chromium/' \
                 'perf/dashboard/ui/changelog.html?url=/trunk/src&range=%d:%d'
 
 # DEPS file URL.
 DEPS_FILE= 'http://src.chromium.org/viewvc/chrome/trunk/src/DEPS?revision=%d'
 
 # WebKit Changelogs URL.
 WEBKIT_CHANGELOG_URL = 'http://trac.webkit.org/log/' \
                        'trunk/?rev=%d&stop_rev=%d&verbose=on'
 
+DONE_MESSAGE = 'You are probably looking for a change made after ' \
+               '%d (known good), but no later than %d (first known bad).'
+
 ###############################################################################
 
 import math
 import optparse
 import os
 import pipes
 import re
 import shutil
 import subprocess
 import sys
@@ skipping 493 matching lines @@
   try:
     last_known_good_webkit_rev = GetWebKitRevisionForChromiumRevision(
         last_known_good_rev)
     first_known_bad_webkit_rev = GetWebKitRevisionForChromiumRevision(
         first_known_bad_rev)
   except Exception, e:
     # Silently ignore the failure.
     last_known_good_webkit_rev, first_known_bad_webkit_rev = 0, 0
 
   # We're done. Let the user know the results in an official manner.
-  print('You are probably looking for build %d.' % first_known_bad_rev)
+  print DONE_MESSAGE % (last_known_good_rev, first_known_bad_rev)
+  print 'All Chromium changes on one page:'
+  print '  ' + CHANGELOG_URL % (last_known_good_rev, first_known_bad_rev)
   if last_known_good_webkit_rev != first_known_bad_webkit_rev:
-    print 'WEBKIT CHANGELOG URL:'
-    print WEBKIT_CHANGELOG_URL % (first_known_bad_webkit_rev,
-                                  last_known_good_webkit_rev)
-  print 'CHANGELOG URL:'
-  print CHANGELOG_URL % (last_known_good_rev, first_known_bad_rev)
-  print 'Built at revision:'
-  print BUILD_VIEWVC_URL % first_known_bad_rev
+    print 'All WebKit changes on one page:'
+    print '  ' + WEBKIT_CHANGELOG_URL % (first_known_bad_webkit_rev,
+                                         last_known_good_webkit_rev)
+  print 'All possible Chromium revisions to investigate:'
+  for rev in range(last_known_good_rev + 1, first_known_bad_rev + 1):
+    print '  ' + BUILD_VIEWVC_URL % rev

[Nico, 2012/04/17 22:04:53]

The scrollback from this will push the more interesting all-in-one-page urls too
far up I think.

 
 
 if __name__ == '__main__':
   sys.exit(main())