[recipes.py] move doc arg parsing to its module

recipes.py

 #!/usr/bin/env python
 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 """Tool to interact with recipe repositories.
 
 This tool operates on the nearest ancestor directory containing an
 infra/config/recipes.cfg.
 """
@@ skipping 223 matching lines @@
 
     if v.get(project_id):
       raise ValueError('An override is already defined for [%s] (%s)' % (
                        project_id, v[project_id]))
     path = os.path.abspath(os.path.expanduser(path))
     if not os.path.isdir(path):
       raise ValueError('Override path [%s] is not a directory' % (path,))
     v[project_id] = path
 
 
-def doc(config_file, package_deps, args):
-  from recipe_engine import doc
-  from recipe_engine import loader
-
-  universe = loader.RecipeUniverse(package_deps, config_file)
-  universe_view = loader.UniverseView(universe, package_deps.root_package)
-
-  doc.main(universe_view, args.recipe, args.kind)
-
-
 # Map of arguments_pb2.Property "value" oneof conversion functions.
 #
 # The fields here should be kept in sync with the "value" oneof field names in
 # the arguments_pb2.Arguments.Property protobuf message.
 _OP_PROPERTY_CONV = {
     's': lambda prop: prop.s,
     'int': lambda prop: prop.int,
     'uint': lambda prop: prop.uint,
     'd': lambda prop: prop.d,
     'b': lambda prop: prop.b,
@@ skipping 100 matching lines @@
   return post_process_args
 
 
 def main():
   parser = argparse.ArgumentParser(
       description='Interact with the recipe system.')
 
   common_postprocess_func = add_common_args(parser)
 
   from recipe_engine import fetch, lint_test, bundle, depgraph, autoroll
-  from recipe_engine import remote, refs
-  to_add = [fetch, lint_test, bundle, depgraph, autoroll, remote, refs]
+  from recipe_engine import remote, refs, doc
+  to_add = [fetch, lint_test, bundle, depgraph, autoroll, remote, refs, doc]
 
   subp = parser.add_subparsers()
   for module in to_add:
     module.add_subparser(subp)
 
   test_p = subp.add_parser(
     'test',
     description='Generate or check expectations by simulation')
   test_p.set_defaults(command='test')
   test_p.add_argument('args', nargs=argparse.REMAINDER)
@@ skipping 62 matching lines @@
       'recipe',
       help='The recipe to execute')
   run_p.add_argument(
       'props',
       nargs=argparse.REMAINDER,
       type=parse_prop,
       help='A list of property pairs; e.g. mastername=chromium.linux '
            'issue=12345. The property value will be decoded as JSON, but if '
            'this decoding fails the value will be interpreted as a string.')
 
-  doc_kinds=('binarypb', 'jsonpb', 'textpb', 'markdown(github)',
-             'markdown(gitiles)')
-  doc_p = subp.add_parser(
-      'doc',
-      description='List all known modules reachable from the current package, '
-          'with their documentation')
-  doc_p.add_argument('recipe', nargs='?',
-                     help='Restrict documentation to this recipe')
-  doc_p.add_argument('--kind', default='jsonpb', choices=doc_kinds,
-                     help='Output this kind of documentation')
-  doc_p.set_defaults(command='doc')
-
   args = parser.parse_args()
   common_postprocess_func(parser, args)
   if hasattr(args, 'postprocess_func'):
     args.postprocess_func(parser, args)
 
   # TODO(iannucci): We should always do logging.basicConfig() (probably with
   # logging.WARNING), even if no verbose is passed. However we need to be
   # careful as this could cause issues with spurious/unexpected output. I think
   # it's risky enough to do in a different CL.
 
@@ skipping 102 matching lines @@
     # an infra failure, rather than a test failure.
     return 2
 
   if hasattr(args, 'func'):
     return args.func(package_deps, args)
 
   if args.command == 'test':
     return test(config_file, package_deps, args)
   elif args.command == 'run':
     return run(config_file, package_deps, args)
-  elif args.command == 'doc':
-    return doc(config_file, package_deps, args)
   else:
     print """Dear sir or madam,
         It has come to my attention that a quite impossible condition has come
     to pass in the specification you have issued a request for us to fulfill.
     It is with a heavy heart that I inform you that, at the present juncture,
     there is no conceivable next action to be taken upon your request, and as
     such, we have decided to abort the request with a nonzero status code.  We
     hope that your larger goals have not been put at risk due to this
     unfortunate circumstance, and wish you the best in deciding the next action
     in your venture and larger life.
@@ skipping 19 matching lines @@
 
   if not isinstance(ret, int):
     if ret is None:
       ret = 0
     else:
       print >> sys.stderr, ret
       ret = 1
   sys.stdout.flush()
   sys.stderr.flush()
   os._exit(ret)