run_isolated: support non-isolated commands

client/isolateserver.py

 #!/usr/bin/env python
 # Copyright 2013 The LUCI Authors. All rights reserved.
 # Use of this source code is governed by the Apache v2.0 license that can be
 # found in the LICENSE file.
 
 """Archives a set of files or directories to an Isolate Server."""
 
 __version__ = '0.4.7'
 
 import base64
@@ skipping 2162 matching lines @@
       '-I', '--isolate-server',
       metavar='URL', default=os.environ.get('ISOLATE_SERVER', ''),
       help='URL of the Isolate Server to use. Defaults to the environment '
            'variable ISOLATE_SERVER if set. No need to specify https://, this '
            'is assumed.')
   parser.add_option(
       '--namespace', default='default-gzip',
       help='The namespace to use on the Isolate Server, default: %default')
 
 
-def process_isolate_server_options(parser, options, set_exception_handler):
-  """Processes the --isolate-server option and aborts if not specified.
+def process_isolate_server_options(
+      parser, options, set_exception_handler, required=True):

[M-A Ruel, 2016/05/02 14:18:14]

Can you not make it a default arg and update all call sites and bump version?

[nodir, 2016/05/02 17:43:22]

Done in a separate patchset

+  """Processes the --isolate-server option.
 
   Returns the identity as determined by the server.
   """
   if not options.isolate_server:
-    parser.error('--isolate-server is required.')
+    if required:
+      parser.error('--isolate-server is required.')
+    return
+
   try:
     options.isolate_server = net.fix_url(options.isolate_server)
   except ValueError as e:
     parser.error('--isolate-server %s' % e)
   if set_exception_handler:
     on_error.report_on_exception_exit(options.isolate_server)
   try:
     return auth.ensure_logged_in(options.isolate_server)
   except ValueError as e:
     parser.error(str(e))
@@ skipping 61 matching lines @@
 def main(args):
   dispatcher = subcommand.CommandDispatcher(__name__)
   return dispatcher.execute(OptionParserIsolateServer(), args)
 
 
 if __name__ == '__main__':
   fix_encoding.fix_encoding()
   tools.disable_buffering()
   colorama.init()
   sys.exit(main(sys.argv[1:]))