Add optparse custom version to prevent unnecessary loading of libtorrent

deluge/main.py

@@ -48,37 +48,40 @@ import deluge.common
 import deluge.configmanager
 import deluge.error
 
+def version_callback(option, opt_str, value, parser):
+    print os.path.basename(sys.argv[0]) + ": " + deluge.common.get_version()
     try:
         from deluge._libtorrent import lt
-    lt_version = "\nlibtorrent: %s" % lt.version
+        print "libtorrent: %s" % lt.version
     except ImportError:
-    lt_version = ""
+        pass
+    raise SystemExit
 
 def start_ui():
     """Entry point for ui script"""
     import deluge.common
 
     # Setup the argument parser
-    parser = OptionParser(usage="%prog [options] [actions]",
-                  version= "%prog: " + deluge.common.get_version() + lt_version)
-
+    parser = OptionParser(usage="%prog [options] [actions]")
+    parser.add_option("-v", "--version", action="callback", callback=version_callback,
+        help="Show program's version number and exit")
     parser.add_option("-u", "--ui", dest="ui",
         help="""The UI that you wish to launch.  The UI choices are:\n
         \t gtk -- A GTK-based graphical user interface (default)\n
         \t web -- A web-based interface (http://localhost:8112)\n
         \t console -- A console or command-line interface""", action="store", type="str")
+    parser.add_option("-s", "--set-default-ui", dest="default_ui",
+        help="Sets the default UI to be run when no UI is specified", action="store", type="str")
+    parser.add_option("-a", "--args", dest="args",
+        help="Arguments to pass to UI, -a '--option args'", action="store", type="str")
     parser.add_option("-c", "--config", dest="config",
         help="Set the config folder location", action="store", type="str")
     parser.add_option("-l", "--logfile", dest="logfile",
         help="Output to designated logfile instead of stdout", action="store", type="str")
-    parser.add_option("-a", "--args", dest="args",
-        help="Arguments to pass to UI, -a '--option args'", action="store", type="str")
     parser.add_option("-L", "--loglevel", dest="loglevel",
         help="Set the log level: none, info, warning, error, critical, debug", action="store", type="str")
     parser.add_option("-q", "--quiet", dest="quiet",
         help="Sets the log level to 'none', this is the same as `-L none`", action="store_true", default=False)
-    parser.add_option("-s", "--set-default-ui", dest="default_ui",
-        help="Sets the default UI to be run when no UI is specified", action="store", type="str")
 
     # Get the options and args from the OptionParser
     (options, args) = parser.parse_args()
@@ -110,6 +113,9 @@ def start_ui():
     if options.quiet:
         options.loglevel = "none"
 
+    if options.loglevel:
+        options.loglevel = options.loglevel.lower()
+
     # Setup the logger
     deluge.log.setupLogger(level=options.loglevel, filename=options.logfile)
 
@@ -134,8 +140,9 @@ def start_daemon():
         warnings.filterwarnings('ignore', category=DeprecationWarning, module='twisted')
 
     # Setup the argument parser
-    parser = OptionParser(usage="%prog [options] [actions]",
-                  version= "%prog: " + deluge.common.get_version() + lt_version)
+    parser = OptionParser(usage="%prog [options] [actions]")
+    parser.add_option("-v", "--version", action="callback", callback=version_callback,
+            help="Show program's version number and exit")
     parser.add_option("-p", "--port", dest="port",
         help="Port daemon will listen on", action="store", type="int")
     parser.add_option("-i", "--interface", dest="listen_interface",
@@ -149,10 +156,10 @@ this should be an IP address", metavar="IFACE",
         help="Do not daemonize", action="store_true", default=False)
     parser.add_option("-c", "--config", dest="config",
         help="Set the config location", action="store", type="str")
-    parser.add_option("-l", "--logfile", dest="logfile",
-        help="Set the logfile location", action="store", type="str")
     parser.add_option("-P", "--pidfile", dest="pidfile",
         help="Use pidfile to store process id", action="store", type="str")
+    parser.add_option("-l", "--logfile", dest="logfile",
+        help="Set the logfile location", action="store", type="str")
     parser.add_option("-L", "--loglevel", dest="loglevel",
         help="Set the log level: none, info, warning, error, critical, debug", action="store", type="str")
     parser.add_option("-q", "--quiet", dest="quiet",

deluge/ui/ui.py

@@ -33,17 +33,22 @@
 #
 #
 
+import os
+import sys
 from optparse import OptionParser, OptionGroup
+
 import deluge.common
 import deluge.configmanager
 import deluge.log
-import os
 
+def version_callback(option, opt_str, value, parser):
+    print os.path.basename(sys.argv[0]) + ": " + deluge.common.get_version()
     try:
         from deluge._libtorrent import lt
-    lt_version = "\nlibtorrent: %s" % lt.version
+        print "libtorrent: %s" % lt.version
     except ImportError:
-    lt_version = ""
+        pass
+    raise SystemExit
 
 DEFAULT_PREFS = {
     "default_ui": "gtk"
@@ -58,10 +63,9 @@ class _UI(object):
     def __init__(self, name="gtk"):
         self.__name = name
 
-        usage="%prog [options] [actions]",
-
-        self.__parser = OptionParser(version="%prog: " + deluge.common.get_version() + lt_version)
-
+        self.__parser = OptionParser(usage="%prog [options] [actions]")
+        self.__parser.add_option("-v", "--version", action="callback", callback=version_callback,
+            help="Show program's version number and exit")
         group = OptionGroup(self.__parser, "Common Options")
         group.add_option("-c", "--config", dest="config",
             help="Set the config folder location", action="store", type="str")