Added implicit empty password to 'presubmit_support.py'

presubmit_support.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.
 
 """Enables directory-specific presubmit checks to run at upload and/or commit.
 """
 
 __version__ = '1.8.0'
 
@@ skipping 1544 matching lines @@
     logging.basicConfig(level=logging.INFO)
   else:
     logging.basicConfig(level=logging.ERROR)
 
   if options.rietveld_email and options.rietveld_email_file:
     parser.error("Only one of --rietveld_email or --rietveld_email_file "
                  "can be passed to this program.")
   if options.rietveld_private_key_file and options.rietveld_password:
     parser.error("Only one of --rietveld_private_key_file or "
                  "--rietveld_password can be passed to this program.")
+
   if options.rietveld_email_file:
     with open(options.rietveld_email_file, "rb") as f:
       options.rietveld_email = f.read().strip()
 
   change_class, files = load_files(options, args)
   if not change_class:
     parser.error('For unversioned directory, <files> is not optional.')
   logging.info('Found %d file(s).' % len(files))
 
   rietveld_obj = None
   if options.rietveld_url:
     # The empty password is permitted: '' is not None.
-    if options.rietveld_password is not None:
-      rietveld_obj = rietveld.CachingRietveld(
-        options.rietveld_url,
-        options.rietveld_email,
-        options.rietveld_password)
-    elif options.rietveld_private_key_file:
+    if options.rietveld_private_key_file:
       rietveld_obj = rietveld.JwtOAuth2Rietveld(
         options.rietveld_url,
         options.rietveld_email,
         options.rietveld_private_key_file)
     else:
-      parser.error("No password or secret key has been provided for "
-                   "Rietveld. Unable to connect.")
+      rietveld_obj = rietveld.CachingRietveld(
+        options.rietveld_url,
+        options.rietveld_email,
+        options.rietveld_password)
     if options.rietveld_fetch:
       assert options.issue
       props = rietveld_obj.get_issue_properties(options.issue, False)
       options.author = props['owner_email']
       options.description = props['description']
       logging.info('Got author: "%s"', options.author)
       logging.info('Got description: """\n%s\n"""', options.description)
   if options.trybot_json:
     with open(options.trybot_json, 'w') as f:
       # Python's sets aren't JSON-encodable, so we convert them to lists here.
@@ skipping 44 matching lines @@
   except PresubmitFailure, e:
     print >> sys.stderr, e
     print >> sys.stderr, 'Maybe your depot_tools is out of date?'
     print >> sys.stderr, 'If all fails, contact maruel@'
     return 2
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   sys.exit(Main(None))