Do not fetch the CL description when using git cl status --fast.

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."""
 
 import datetime
@@ skipping 680 matching lines @@
     if not self.GetIssue():
       return None
     return '%s/%s' % (self.GetRietveldServer(), self.GetIssue())
 
   def GetDescription(self, pretty=False):
     if not self.has_description:
       if self.GetIssue():
         issue = self.GetIssue()
         try:
           self.description = self.RpcServer().get_description(issue).strip()
-        except urllib2.HTTPError, e:
+        except urllib2.HTTPError as e:
           if e.code == 404:
             DieWithError(
                 ('\nWhile fetching the description for issue %d, received a '
                  '404 (not found)\n'
                  'error. It is likely that you deleted this '
                  'issue on the server. If this is the\n'
                  'case, please run\n\n'
                  '    git cl issue 0\n\n'
                  'to clear the association with the deleted issue. Then run '
                  'this command again.') % issue)
           else:
             DieWithError(
                 '\nFailed to fetch issue description. HTTP error %d' % e.code)
+        except urllib2.URLError as e:
+          print >> sys.stderr, (
+              'Warning: Failed to retrieve CL description due to network '
+              'failure.')
+          self.description = ''
+
       self.has_description = True
     if pretty:
       wrapper = textwrap.TextWrapper()
       wrapper.initial_indent = wrapper.subsequent_indent = '  '
       return wrapper.fill(self.description)
     return self.description
 
   def GetPatchset(self):
     """Returns the patchset number as a int or None if not set."""
     if self.patchset is None and not self.lookedup_patchset:
@@ skipping 584 matching lines @@
           alignment, ShortBranchName(branch), color, issue, reset)
 
   cl = Changelist()
   print
   print 'Current branch:',
   if not cl.GetIssue():
     print 'no issue assigned.'
     return 0
   print cl.GetBranch()
   print 'Issue number: %s (%s)' % (cl.GetIssue(), cl.GetIssueURL())
-  print 'Issue description:'
-  print cl.GetDescription(pretty=True)
+  if not options.fast:
+    print 'Issue description:'

[iannucci, 2014/07/27 18:05:09]

it would be neat to cache this non-fast stuff. It doesn't really change that
often, and it's a useful summary to have (for people who have lots of branches).

[M-A Ruel, 2014/07/28 15:35:41]

The funny thing is that gcl does that. It should be added to git-cl one day.

+    print cl.GetDescription(pretty=True)
   return 0
 
 
 def colorize_CMDstatus_doc():
   """To be called once in main() to add colors to git cl status help."""
   colors = [i for i in dir(Fore) if i[0].isupper()]
 
   def colorize_line(line):
     for color in colors:
       if color in line.upper():
@@ skipping 1318 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:]))