Make git cl status spit out URLs instead of just Rietveld issue numbers.

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 difflib
@@ skipping 587 matching lines @@
       # with that branch.
       if self.GetIssue():
         self.rietveld_server = gclient_utils.UpgradeToHttps(RunGit(
             ['config', self._RietveldServer()], error_ok=True).strip())
       if not self.rietveld_server:
         self.rietveld_server = settings.GetDefaultServerUrl()
     return self.rietveld_server
 
   def GetIssueURL(self):
     """Get the URL for a particular issue."""
+    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:
           if e.code == 404:
@@ skipping 422 matching lines @@
   parser.add_option('--field',
                     help='print only specific field (desc|id|patch|url)')
   (options, args) = parser.parse_args(args)
 
   # TODO: maybe make show_branches a flag if necessary.
   show_branches = not options.field
 
   if show_branches:
     branches = RunGit(['for-each-ref', '--format=%(refname)', 'refs/heads'])
     if branches:
+      changes = (Changelist(branchref=b) for b in branches.splitlines())
+      branches = dict((cl.GetBranch(), cl.GetIssueURL()) for cl in changes)
+      alignment = max(5, max(len(b) for b in branches))
       print 'Branches associated with reviews:'
-      changes = (Changelist(branchref=b) for b in branches.splitlines())
-      branches = dict((cl.GetBranch(), cl.GetIssue()) for cl in changes)
-      alignment = max(5, max(len(b) for b in branches))
       for branch in sorted(branches):
         print "  %*s: %s" % (alignment, branch, branches[branch])

[M-A Ruel, 2013/06/18 18:48:45]

if branches[branch]:
...
else:
don't print the branch

[Dan Beam, 2013/06/18 18:55:01]

Done.

 
   cl = Changelist()
   if options.field:
     if options.field.startswith('desc'):
       print cl.GetDescription()
     elif options.field == 'id':
       issueid = cl.GetIssue()
       if issueid:
         print issueid
     elif options.field == 'patch':
@@ skipping 1074 matching lines @@
   GenUsage(parser, 'help')
   return CMDhelp(parser, argv)
 
 
 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:]))