From: david-sarah <david-sarah@jacaranda.org>
Date: Fri, 21 Jan 2011 07:59:13 +0000 (-0800)
Subject: Change --version and --version-and-path to not exit immediately, if a command is... 
X-Git-Tag: allmydata-tahoe-1.8.2b1~20
X-Git-Url: https://git.rkrishnan.org/pf/content//%22%22?a=commitdiff_plain;h=28bd80812f9b81f5eccf3abcfd885a353d3fdf56;p=tahoe-lafs%2Ftahoe-lafs.git

Change --version and --version-and-path to not exit immediately, if a command is given.
---

diff --git a/src/allmydata/scripts/common.py b/src/allmydata/scripts/common.py
index 5eb31477..4fa217ca 100644
--- a/src/allmydata/scripts/common.py
+++ b/src/allmydata/scripts/common.py
@@ -32,8 +32,8 @@ class BaseOptions(usage.Options):
 
     optFlags = [
         ["quiet", "q", "Operate silently."],
-        ["version", "V", "Display version numbers and exit."],
-        ["version-and-path", None, "Display version numbers and paths to their locations and exit."],
+        ["version", "V", "Display version numbers."],
+        ["version-and-path", None, "Display version numbers and paths to their locations."],
     ]
     optParameters = [
         ["node-directory", "d", None, "Specify which Tahoe node directory should be used." + (
@@ -43,12 +43,14 @@ class BaseOptions(usage.Options):
     def opt_version(self):
         import allmydata
         print >>self.stdout, allmydata.get_package_versions_string()
-        sys.exit(0)
+        print >>self.stdout
+        self.no_command_needed = True
 
     def opt_version_and_path(self):
         import allmydata
         print >>self.stdout, allmydata.get_package_versions_string(show_paths=True)
-        sys.exit(0)
+        print >>self.stdout
+        self.no_command_needed = True
 
 
 class BasedirMixin:
diff --git a/src/allmydata/scripts/runner.py b/src/allmydata/scripts/runner.py
index 1cd8a712..50f2f07a 100644
--- a/src/allmydata/scripts/runner.py
+++ b/src/allmydata/scripts/runner.py
@@ -35,7 +35,9 @@ class Options(BaseOptions, usage.Options):
 
     def postOptions(self):
         if not hasattr(self, 'subOptions'):
-            raise usage.UsageError("must specify a command")
+            if not hasattr(self, 'no_command_needed'):
+                raise usage.UsageError("must specify a command")
+            sys.exit(0)
 
 
 create_dispatch = {}