Add git cl comments to read rietveld comments without having to use the rietveld UI

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 json
@@ skipping 850 matching lines @@
     try:
       issue = int(args[0])
     except ValueError:
       DieWithError('Pass a number to set the issue or none to list it.\n'
           'Maybe you want to run git cl status?')
     cl.SetIssue(issue)
   print 'Issue number:', cl.GetIssue(), '(%s)' % cl.GetIssueURL()
   return 0
 
 
+def CMDcomments(parser, args):
+  """show review comments of the current changelist"""
+  (_, args) = parser.parse_args(args)
+  if args:
+    parser.error('Unsupported argument: %s' % args)
+
+  cl = Changelist()
+  if cl.GetIssue():
+    data = cl.RpcServer().get_issue_properties(cl.GetIssue(), True)
+    for message in sorted(data['messages'], key=lambda x: x['date']):
+      print '\n%s  %s' % (message['date'].split('.', 1)[0], message['sender'])
+      if message['text'].strip():
+        print '\n'.join('  ' + l for l in message['text'].splitlines())
+  return 0
+
+
 def CreateDescriptionFromLog(args):
   """Pulls out the commit log to use as a base for the CL description."""
   log_args = []
   if len(args) == 1 and not args[0].endswith('.'):
     log_args = [args[0] + '..']
   elif len(args) == 1 and args[0].endswith('...'):
     log_args = [args[0][:-1]]
   elif len(args) == 2:
     log_args = [args[0] + '..' + args[1]]
   else:
@@ skipping 194 matching lines @@
                       help='target branch to upload')
   (options, args) = parser.parse_args(args)
 
   # Print warning if the user used the -m/--message argument.  This will soon
   # change to -t/--title.
   if options.message:
     print >> sys.stderr, (
         '\nWARNING: Use -t or --title to set the title of the patchset.\n'
         'In the near future, -m or --message will send a message instead.\n'
         'See http://goo.gl/JGg0Z for details.\n')
-    
+
   # Make sure index is up-to-date before running diff-index.
   RunGit(['update-index', '--refresh', '-q'], error_ok=True)
   if RunGit(['diff-index', 'HEAD']):
     print 'Cannot upload with a dirty tree.  You must commit locally first.'
     return 1
 
   cl = Changelist()
   if args:
     # TODO(ukai): is it ok for gerrit case?
     base_branch = args[0]
@@ skipping 458 matching lines @@
              'and retry or visit go/isgaeup.\n%s') % (e.code, str(e)))
 
   # Not a known command. Default to help.
   GenUsage(parser, 'help')
   return CMDhelp(parser, argv)
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   sys.exit(main(sys.argv[1:]))