samba-tool: add -H or --URL where necessary

To improve consistency, I've made sure all the commands take either a -H
or --URL when specifying a URL

Signed-off-by: Andrew Tridgell <tridge@samba.org>

diff --git a/source4/scripting/python/samba/netcmd/dbcheck.py b/source4/scripting/python/samba/netcmd/dbcheck.py
index 299879b2c642e90906fd738ab66ad51c94374b08..3f379c0cd4d4ea5733b864d2ed740785aaff870a 100644 (file)
--- a/source4/scripting/python/samba/netcmd/dbcheck.py
+++ b/source4/scripting/python/samba/netcmd/dbcheck.py
@@ -52,7 +52,8 @@ class cmd_dbcheck(Command):
             help="don't print details of checking"),
         Option("--attrs", dest="attrs", default=None, help="list of attributes to check (space separated)"),
         Option("--reindex", dest="reindex", default=False, action="store_true", help="force database re-index"),
-        Option("-H", help="LDB URL for database or target server (defaults to local SAM database)", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server (defaults to local SAM database)",
+               type=str, metavar="URL", dest="H"),
         ]
 
     def run(self, DN=None, H=None, verbose=False, fix=False, yes=False, cross_ncs=False, quiet=False,

diff --git a/source4/scripting/python/samba/netcmd/domain.py b/source4/scripting/python/samba/netcmd/domain.py
index 680cfe9d10b316adf47c561a8f775dccc83f18d4..8dffbd2b566ea42127654c8c9d95f2b40511cc65 100644 (file)
--- a/source4/scripting/python/samba/netcmd/domain.py
+++ b/source4/scripting/python/samba/netcmd/domain.py
@@ -119,7 +119,8 @@ class cmd_domain_level(Command):
     synopsis = "%prog domain level (show | raise <options>)"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--quiet", help="Be quiet", action="store_true"),
         Option("--forest", type="choice", choices=["2003", "2008", "2008_R2"],
             help="The forest function level (2003 | 2008 | 2008_R2)"),
@@ -335,7 +336,8 @@ class cmd_domain_passwordsettings(Command):
     synopsis = "%prog domain passwordsettings (show | set <options>)"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--quiet", help="Be quiet", action="store_true"),
         Option("--complexity", type="choice", choices=["on","off","default"],
           help="The password complexity (on | off | default). Default is 'on'"),

diff --git a/source4/scripting/python/samba/netcmd/dsacl.py b/source4/scripting/python/samba/netcmd/dsacl.py
index 5eee235b41ba772e45efc2fd8f2f6c5af5a37f93..f4f33561b54c0a50c331b1ccec7df99cb2bcf559 100644 (file)
--- a/source4/scripting/python/samba/netcmd/dsacl.py
+++ b/source4/scripting/python/samba/netcmd/dsacl.py
@@ -52,8 +52,8 @@ class cmd_ds_acl_set(Command):
     car_help = """ The access control right to allow or deny """
 
     takes_options = [
-        Option("--host", help="LDB URL for database or target server",
-            type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server",
+               type=str, metavar="URL", dest="H"),
         Option("--car", type="choice", choices=["change-rid",
                                                 "change-pdc",
                                                 "change-infrastructure",
@@ -131,7 +131,7 @@ class cmd_ds_acl_set(Command):
         print desc_sddl
 
     def run(self, car, action, objectdn, trusteedn, sddl,
-            host=None, credopts=None, sambaopts=None, versionopts=None):
+            H=None, credopts=None, sambaopts=None, versionopts=None):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp)
 
@@ -139,7 +139,7 @@ class cmd_ds_acl_set(Command):
                              or objectdn is None or trusteedn is None):
             return self.usage()
 
-        samdb = SamDB(url=host, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
         cars = {'change-rid' : GUID_DRS_CHANGE_RID_MASTER,
                 'change-pdc' : GUID_DRS_CHANGE_PDC,

diff --git a/source4/scripting/python/samba/netcmd/fsmo.py b/source4/scripting/python/samba/netcmd/fsmo.py
index aaf7d138e6048ab5e8bf9410a36482197fe41578..d709cc8fda5f70f38763ccb2a41bacca32881531 100644 (file)
--- a/source4/scripting/python/samba/netcmd/fsmo.py
+++ b/source4/scripting/python/samba/netcmd/fsmo.py
@@ -38,7 +38,8 @@ class cmd_fsmo(Command):
     synopsis = "(show | transfer <options> | seize <options>)"
 
     takes_options = [
-        Option("--url", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--force", help="Force seizing of the role without attempting to transfer first.", action="store_true"),
         Option("--role", type="choice", choices=["rid", "pdc", "infrastructure","schema","naming","all"],
                help="""The FSMO role to seize or transfer.\n
@@ -132,12 +133,12 @@ all=all of the above"""),
         print("FSMO transfer of '%s' role successful" % role)
 
 
-    def run(self, subcommand, force=None, url=None, role=None,
+    def run(self, subcommand, force=None, H=None, role=None,
             credopts=None, sambaopts=None, versionopts=None):
         lp = sambaopts.get_loadparm()
         creds = credopts.get_credentials(lp, fallback_machine=True)
 
-        samdb = SamDB(url=url, session_info=system_session(),
+        samdb = SamDB(url=H, session_info=system_session(),
             credentials=creds, lp=lp)
 
         domain_dn = samdb.domain_dn()

diff --git a/source4/scripting/python/samba/netcmd/gpo.py b/source4/scripting/python/samba/netcmd/gpo.py
index 2a6bef0f37f5be96fc01988797c8fbb846eb0658..1301548b67c6b9683908012faf0152e698d4a307 100644 (file)
--- a/source4/scripting/python/samba/netcmd/gpo.py
+++ b/source4/scripting/python/samba/netcmd/gpo.py
@@ -90,7 +90,8 @@ class cmd_listall(Command):
     synopsis = "%prog gpo listall"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str)
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H")
         ]
 
     def run(self, H=None, sambaopts=None,
@@ -134,7 +135,8 @@ class cmd_list(Command):
     takes_args = [ 'username' ]
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str)
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H")
         ]
 
     def run(self, username, H=None, sambaopts=None,

diff --git a/source4/scripting/python/samba/netcmd/group.py b/source4/scripting/python/samba/netcmd/group.py
index 7f3b6cdef9abd47c53cf104e4eb482a0c8f3b44d..df8cf7c87a5a7b0fafc96addeb41839034dfaade 100644 (file)
--- a/source4/scripting/python/samba/netcmd/group.py
+++ b/source4/scripting/python/samba/netcmd/group.py
@@ -46,7 +46,8 @@ class cmd_group_add(Command):
     synopsis = "%prog group add [options] <groupname>"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--groupou",
           help="Alternative location (without domainDN counterpart) to default CN=Users in which new user object will be created",
           type=str),
@@ -89,7 +90,8 @@ class cmd_group_delete(Command):
     synopsis = "%prog group delete <groupname>"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
     ]
 
     takes_args = ["groupname"]
@@ -114,7 +116,8 @@ class cmd_group_add_members(Command):
     synopsis = "%prog group addmembers <groupname> <listofmembers>"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
     ]
 
     takes_args = ["groupname", "listofmembers"]
@@ -140,7 +143,8 @@ class cmd_group_remove_members(Command):
     synopsis = "%prog group removemembers <groupname> <listofmembers>"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
     ]
 
     takes_args = ["groupname", "listofmembers"]

diff --git a/source4/scripting/python/samba/netcmd/user.py b/source4/scripting/python/samba/netcmd/user.py
index 3b772de87a115ab1006382e107667e5067d2101f..1b65777ce14b4017ec9e3334c96c4344c78219bf 100644 (file)
--- a/source4/scripting/python/samba/netcmd/user.py
+++ b/source4/scripting/python/samba/netcmd/user.py
@@ -83,7 +83,8 @@ class cmd_user_enable(Command):
     synopsis = "%prog user enable (<username>|--filter <filter>) [options]"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str, metavar="URL"),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--filter", help="LDAP Filter to set password on", type=str),
         ]
 
@@ -116,7 +117,8 @@ class cmd_user_setexpiry(Command):
     synopsis = "%prog user setexpiry (<username>|--filter <filter>) [options]"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str, metavar="URL"),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--filter", help="LDAP Filter to set password on", type=str),
         Option("--days", help="Days to expiry", type=int, default=0),
         Option("--noexpiry", help="Password does never expire", action="store_true", default=False),
@@ -151,7 +153,8 @@ class cmd_user_setpassword(Command):
     synopsis = "%prog user setpassword (<username>|--filter <filter>) [options]"
 
     takes_options = [
-        Option("-H", help="LDB URL for database or target server", type=str, metavar="URL"),
+        Option("-H", "--URL", help="LDB URL for database or target server", type=str,
+               metavar="URL", dest="H"),
         Option("--filter", help="LDAP Filter to set password on", type=str),
         Option("--newpassword", help="Set password", type=str),
         Option("--must-change-at-next-login",