commit ed5ab1d3e06b93c1332b5fcfc8498229ff8815c1
author Mike Frysinger <vapier@chromium.org> Mon Oct 23 14:10:28 2017 -0400
committer chrome-bot <chrome-bot@chromium.org> Fri Dec 15 15:52:45 2017 -0800
tree 2b0bf59bb6a0beeef568b8647dc9b9ad88cca70c
parent cd0856953252117831f4c4441093924d390ae14d

cros_workon: move parser into a dedicated func

This makes the main func easier to follow.

BUG=None
TEST=`cros_workon ...` still works

Change-Id: I76046ff25494ff35d80748aa4cd2af7f17a5b3b5
Reviewed-on: https://chromium-review.googlesource.com/827690
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Paul Hobbs <phobbs@google.com>

scripts/cros_workon.py

diff --git a/scripts/cros_workon.py b/scripts/cros_workon.py
index a06e8fb..836d397 100644
--- a/scripts/cros_workon.py
+++ b/scripts/cros_workon.py
@@ -19,7 +19,8 @@
 from chromite.lib import workon_helper
 
 
-def main(argv):
+def GetParser():
+  """Get a CLI parser."""
   parser = commandline.ArgumentParser(description=__doc__)
   parser.add_argument('--board', default=cros_build_lib.GetDefaultBoard(),
                       help='The board to set package keywords for.')
@@ -53,6 +54,11 @@
     sub_parser.add_argument('packages', nargs='*',
                             help='The packages to run command against.')
 
+  return parser
+
+
+def main(argv):
+  parser = GetParser()
   options = parser.parse_args(argv)
   options.Freeze()