[PATCH] Cleanup help message in plague-client

[Josh Boyer]

Hi all,

This patch cleans up the Usage message in plague-client by listing all
of the available commands and moving them into a function.  It also adds
a help command, which I think was asked for by Jef.

thx,
josh

Index: extras-buildsys/client/client.py
===================================================================
--- extras-buildsys.orig/client/client.py
+++ extras-buildsys/client/client.py
@@ -300,22 +300,27 @@ def getEmailAddress():
 
     return config_email
 
-
+def Usage():
+    print "Usage:\npackage-client.py <command>\n"
+    print "      <command> is one of:"
+    print "      build [package_name] [cvs_tag | srpm_path] [target]"
+    print "      list"
+    print "      list <status>"
+    print "      list <email> <status>"
+    print "      kill <jobid>"
+    print "      update_builders"
+    print "      list_builders"
+    print "      pause"
+    print "      unpause"
+    print "      is_paused"
+    print "      requeue <jobid>"
+    print "      detail <jobid>"
+    print "      help"
+    print ""
 
 if __name__ == '__main__':
     if len(sys.argv) < 2:
-        print "Usage:\npackage-builder.py <command>\n"
-        print "      <command> is one of:"
-        print "      build [package_name] [cvs_tag | srpm_path] [target]"
-        print "      list"
-        print "      list <status>"
-        print "      list <email> <status>"
-        print "      kill <jobid>"
-        print "      update_builders"
-        print "      list_builders"
-        print "      pause"
-        print "      unpause"
-	print ""
+        Usage()
         sys.exit(1)
 
     # Write out config file if it doesn't exist
@@ -365,7 +370,11 @@ if __name__ == '__main__':
         pause(server, False)
     elif cmd == 'is_paused':
         is_paused(server)
+    elif cmd == 'help':
+        Usage()
+        sys.exit(1)
     else:
         print "Unknown command."
+        Usage()
         sys.exit(1)