[recipes.py] move depgraph 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 239 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 depgraph(config_file, package_deps, args):
-  from recipe_engine import depgraph
-  from recipe_engine import loader
-
-  universe = loader.RecipeUniverse(package_deps, config_file)
-
-  depgraph.main(universe, package_deps.root_package,
-                args.ignore_package, args.output, args.recipe_filter)
-
-
 def refs(config_file, package_deps, args):
   from recipe_engine import refs
   from recipe_engine import loader
 
   universe = loader.RecipeUniverse(package_deps, config_file)
 
   refs.main(universe, package_deps.root_package, args.modules, args.transitive)
 
 
 def doc(config_file, package_deps, args):
@@ skipping 113 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
-  to_add = [fetch, lint_test, bundle]
+  from recipe_engine import fetch, lint_test, bundle, depgraph
+  to_add = [fetch, lint_test, bundle, depgraph]
 
   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 102 matching lines @@
   autoroll_p.add_argument(
       '--output-json',
       type=os.path.abspath,
       help='A json file to output information about the roll to.')
   autoroll_p.add_argument(
       '--verbose-json',
       action='store_true',
       help=('Emit even more data in the output-json file. '
             'Requires --output-json.'))
 
-  depgraph_p = subp.add_parser(
-      'depgraph',
-      description=(
-          'Produce graph of recipe and recipe module dependencies. Example: '
-          './recipes.py --package infra/config/recipes.cfg depgraph | tred | '
-          'dot -Tpdf > graph.pdf'))
-  depgraph_p.set_defaults(command='depgraph')
-  depgraph_p.add_argument(
-      '--output', type=argparse.FileType('w'), default=sys.stdout,
-      help='The file to write output to')
-  depgraph_p.add_argument(
-      '--ignore-package', action='append', default=[],
-      help='Ignore a recipe package (e.g. recipe_engine). Can be passed '
-           'multiple times')
-  depgraph_p.add_argument(
-      '--recipe-filter', default='',
-      help='A recipe substring to examine. If present, the depgraph will '
-           'include a recipe section containing recipes whose names contain '
-           'this substring. It will also filter all nodes of the graph to only '
-           'include modules touched by the filtered recipes.')
-
   refs_p = subp.add_parser(
       'refs',
       description='List places referencing given recipe module(s).')
   refs_p.set_defaults(command='refs')
   refs_p.add_argument('modules', nargs='+', help='Module(s) to query for')
   refs_p.add_argument('--transitive', action='store_true',
                       help='Compute transitive closure of the references')
 
   doc_kinds=('binarypb', 'jsonpb', 'textpb', 'markdown(github)',
              'markdown(gitiles)')
@@ skipping 126 matching lines @@
 
   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 == 'autoroll':
     return autoroll(repo_root, config_file, args)
-  elif args.command == 'depgraph':
-    return depgraph(config_file, package_deps, args)
   elif args.command == 'refs':
     return refs(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
@@ skipping 23 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)