[recipes.py] move refs 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 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):
   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)
 
 
@@ skipping 110 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
-  to_add = [fetch, lint_test, bundle, depgraph, autoroll, remote]
+  from recipe_engine import remote, refs
+  to_add = [fetch, lint_test, bundle, depgraph, autoroll, remote, refs]
 
   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.')
 
-  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)')
   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')
@@ skipping 114 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 == '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
     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
@@ skipping 21 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)