Be more user friendly and add optparse error messages.

janl · janl · commit a74ca785e2d9 · 2008-09-11T15:36:44.000Z
--HG--
extra : convert_revision : svn%3A7a298fb0-333a-0410-83e7-658617cd9cf3/trunk%40106
diff --git a/couchdb/tools/replication-helper.py b/couchdb/tools/replication-helper.py
@@ -118,7 +118,10 @@ def __call__(self):
 
 
 def main():
-    parser = optparse.OptionParser(usage='%prog [options]', version=VERSION)
+    usage = '%prog [options] --source-server=http://server:port/ \
+--target-servers=http://server2:port2/[,http://server3:port3/, ...]'
+
+    parser = optparse.OptionParser(usage=usage, version=VERSION)
 
     parser.add_option('--source-server',
         action='store',
@@ -146,9 +149,11 @@ def main():
 
     options, arg = parser.parse_args()
 
-    options.target_servers = options.target_servers.split(',')
     if not options.target_servers or not options.source_server:
+        parser.error("Need at least --source-server and --target-servers")
         sys.exit(1)
+
+    options.target_servers = options.target_servers.split(',')
             
     ReplicationHelper(options)()
 
