Re: [chirp_devel] [PATCH 5/9] Refactor version display into chirp module (#2343)

+class VersionAction(argparse.Action):

• def __call__(self, parser, namespace, value, option_string=None):

•    args = (CHIRP_VERSION,
  

•            platform.get_platform().os_version_string(),
  

•            sys.version.split()[0])
  

•    print "CHIRP %s on %s (Python %s)" % args
  

•    sys.exit(1)
  

It's not okay, IMHO, to exit() from a library call.

Overall, I don't think it's worth having a separate module for just this little thing. I'd just put it in __init__.py, or in chirp_common.

--Dan