Allow the caller to specify a username and password so that private issues

apply_issue.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.
 
 """Applies an issue from Rietveld.
 """
 
 import getpass
 import logging
 import optparse
 import os
 import sys
 import urllib2
 
 import breakpad  # pylint: disable=W0611
 
 import checkout
 import fix_encoding
 import rietveld
 import scm
 
 
 def main():
   parser = optparse.OptionParser(description=sys.modules[__name__].__doc__)
   parser.add_option(
       '-v', '--verbose', action='count', default=0,
       help='Prints debugging infos')
   parser.add_option(
-      '-e',
-      '--email',
-      help='IGNORED: Kept for compatibility.')
+      '-e', '--email', default='',
+      help='Email address to access rietveld.  If not specified, anonymous '
+           'access will be used.')
+  parser.add_option(
+      '-w', '--password', default=None, help='Password for email addressed.')
   parser.add_option(
       '-i', '--issue', type='int', help='Rietveld issue number')
   parser.add_option(
       '-p', '--patchset', type='int', help='Rietveld issue\'s patchset number')
   parser.add_option(
       '-r',
       '--root_dir',
       default=os.getcwd(),
       help='Root directory to apply the patch')
   parser.add_option(
       '-s',
       '--server',
       default='http://codereview.chromium.org',
       help='Rietveld server')
   options, args = parser.parse_args()
   logging.basicConfig(
       format='%(levelname)5s %(module)11s(%(lineno)4d): %(message)s',
       level=[logging.WARNING, logging.INFO, logging.DEBUG][
           min(2, options.verbose)])
   if args:
     parser.error('Extra argument(s) "%s" not understood' % ' '.join(args))
   if not options.issue:
     parser.error('Require --issue')
   options.server = options.server.rstrip('/')
   if not options.server:
     parser.error('Require a valid server')
 
-  obj = rietveld.Rietveld(options.server, '', None)
+  obj = rietveld.Rietveld(options.server, options.email, options.password)
   try:
     properties = obj.get_issue_properties(options.issue, False)
   except rietveld.upload.ClientLoginError, e:
     if sys.stdout.closed:
       print >> sys.stderr, 'Accessing the issue requires login.'
       return 1
     print('Accessing the issue requires login.')
     obj = rietveld.Rietveld(options.server, None, None)
     properties = obj.get_issue_properties(options.issue, False)
 
@@ skipping 37 matching lines @@
     scm_obj.apply_patch(patchset)
   except checkout.PatchApplicationFailed, e:
     print >> sys.stderr, str(e)
     return 1
   return 0
 
 
 if __name__ == "__main__":
   fix_encoding.fix_encoding()
   sys.exit(main())