Process first branch status request before spawning threads.

git_cl.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.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld."""
 
 from distutils.version import LooseVersion
@@ skipping 1191 matching lines @@
       if url:
         print url
     return 0
 
   branches = RunGit(['for-each-ref', '--format=%(refname)', 'refs/heads'])
   if not branches:
     print('No local branch found.')
     return 0
 
   changes = (Changelist(branchref=b) for b in branches.splitlines())
-  branches = dict((c.GetBranch(), c.GetIssueURL()) for c in changes)
+  branches = [c.GetBranch() for c in changes]
   alignment = max(5, max(len(b) for b in branches))
   print 'Branches associated with reviews:'
   # Adhoc thread pool to request data concurrently.
   output = Queue.Queue()
 
   # Silence upload.py otherwise it becomes unweldly.
   upload.verbosity = 0
 
   if not options.fast:
     def fetch(b):
@@ skipping 25 matching lines @@
         color = Fore.GREEN
       elif not msgs:
         # No message was sent.
         color = Fore.RED
       elif msgs[-1]['sender'] != props.get('owner_email'):
         color = Fore.YELLOW
       else:
         color = Fore.BLUE
       output.put((b, i, color))
 
-    threads = [threading.Thread(target=fetch, args=(b,)) for b in branches]
+    # Process one branch synchronously to work through authentication, then
+    # spawn threads to process all the other branches in parallel.
+    if branches:
+      fetch(branches[0])
+    threads = [
+      threading.Thread(target=fetch, args=(b,)) for b in branches[1:]]
     for t in threads:
       t.daemon = True
       t.start()
   else:
     # Do not use GetApprovingReviewers(), since it requires an HTTP request.
     for b in branches:
       c = Changelist(branchref=b)
       url = c.GetIssueURL()
       output.put((b, url, Fore.BLUE if url else Fore.WHITE))
 
@@ skipping 1271 matching lines @@
         ('AppEngine is misbehaving and returned HTTP %d, again. Keep faith '
           'and retry or visit go/isgaeup.\n%s') % (e.code, str(e)))
 
 
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   colorama.init()
   sys.exit(main(sys.argv[1:]))