Changeset 2741 for trunk

Timestamp:

Dec 21, 2009, 12:22:22 AM (15 years ago)

Author:

price

Message:

invirt.fcgi: fix usage message, simplify option-parsing logic

File:

• trunk/packages/invirt-web/code/invirt.fcgi (modified) (1 diff)

trunk/packages/invirt-web/code/invirt.fcgi

@@ -14 +14 @@
     argv0_dir = os.path.dirname(sys.argv[0])
     print >>sys.stderr, """%s <unauth|auth> [config]
-%s/auth.fcgi [config]
-%s/unauth.fcgi [config]
 
 Run server as FastCGI, with CherryPy config from "main.conf".
-
 With `config`, run standalone with CherryPy config from `config`.
 
-Run this script as either 'auth.fcgi' or 'unauth.fcgi', to get
-the authenticated or unauthenticated site respectively, or pass
-'auth' or 'unauth' as the first parameter.
+Serve the authenticated or unauthenticated site according to
+the first argument.
+
+Helper scripts "auth.fcgi" and "unauth.fcgi" are provided to
+facilitate running the server with no arguments.
 """ % (sys.argv[0], argv0_dir, argv0_dir)
     sys.exit(2)
 
 if __name__ == "__main__":
-    if len(sys.argv) > 3 or len(sys.argv) == 1 or '-h' in sys.argv or '--help' in sys.argv:
+    if '-h' in sys.argv or '--help' in sys.argv:
         usage()
+    if not (2 <= len(sys.argv) <= 3):
+        usage()
+
+    mode = sys.argv[1]
     if len(sys.argv) == 3:
-        mode = sys.argv[1]
         conf_file = sys.argv[2]
         dev = True
     else:
-        mode = sys.argv[1]
         conf_file = os.path.join(base_dir, 'main.conf')