From ac180b95d1867c4b4daf9729e97ce93f3201d178 Mon Sep 17 00:00:00 2001
From: Mark McLoughlin <markmc@redhat.com>
Date: Mon, 12 Nov 2012 16:25:55 -0500
Subject: Use stock argparse --version behaviour

optparse prints the version to stdout, argparse prints the version to
stderr.

There's no need to preserve the old optparse behaviour, let's just stick
with argparse behaviour.

Change-Id: Ie141c72112a63149d098afa9db55a95a309e79d7
---
 openstack/common/cfg.py | 35 +----------------------------------
 1 file changed, 1 insertion(+), 34 deletions(-)

(limited to 'openstack')

diff --git a/openstack/common/cfg.py b/openstack/common/cfg.py
index 531c650..d234608 100644
--- a/openstack/common/cfg.py
+++ b/openstack/common/cfg.py
@@ -925,33 +925,6 @@ class MultiConfigParser(object):
         raise KeyError
 
 
-class _VersionAction(argparse.Action):
-
-    def __init__(self,
-                 option_strings,
-                 version=None,
-                 dest=argparse.SUPPRESS,
-                 default=argparse.SUPPRESS,
-                 help="show program's version number and exit"):
-        super(_VersionAction, self).__init__(option_strings=option_strings,
-                                             dest=dest,
-                                             default=default,
-                                             nargs=0,
-                                             help=help)
-        self.version = version
-
-    def __call__(self, parser, namespace, values, option_string=None):
-        version = self.version
-        if version is None:
-            version = parser.version
-        formatter = parser._get_formatter()
-        formatter.add_text(version)
-        message = formatter.format_help()
-        if message:
-            parser._print_message(message, sys.stdout)
-        sys.exit(0)
-
-
 class ConfigCliParser(argparse.ArgumentParser):
 
     def __init__(self, prog=None, usage=None, version=None, *args, **kwargs):
@@ -962,9 +935,7 @@ class ConfigCliParser(argparse.ArgumentParser):
         if usage is not None:
             self.usage = usage.replace("%prog", self.prog)
 
-        self.add_argument('--version',
-                          action=_VersionAction,
-                          version=version)
+        self.add_argument('--version', action='version', version=version)
         self._optionals.title = 'Options'
 
     def add_argument(self, *args, **kwargs):
@@ -1008,10 +979,6 @@ class ConfigCliParser(argparse.ArgumentParser):
             msg = msg.replace("usage:", "Usage:")
         print >>file, msg
 
-    def print_version(self, file=None):
-        super(ConfigCliParser, self)._print_message(self.format_version(),
-                                                    file)
-
 
 class ConfigOpts(collections.Mapping):
 
-- 
cgit