From 43f23b55c44094b29c8a7017fda8e6b030e2eba4 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Wed, 19 Oct 2011 09:37:47 +1100 Subject: [PATCH] s4-s3-upgrade rename samba-tool domain samba3upgrade --libdir to --dbdir for clarity The things pointed at are not typically in a directory called lib, so avoid confusing our administrators. Andrew Bartlett Autobuild-User: Andrew Bartlett Autobuild-Date: Wed Oct 19 15:43:04 CEST 2011 on sn-devel-104 --- .../scripting/python/samba/netcmd/domain.py | 30 +++++++++---------- source4/scripting/python/samba/upgrade.py | 2 +- source4/setup/tests/blackbox_s3upgrade.sh | 4 +-- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/source4/scripting/python/samba/netcmd/domain.py b/source4/scripting/python/samba/netcmd/domain.py index 47b7e4a8dda..715b376657c 100644 --- a/source4/scripting/python/samba/netcmd/domain.py +++ b/source4/scripting/python/samba/netcmd/domain.py @@ -530,7 +530,7 @@ class cmd_domain_passwordsettings(Command): class cmd_domain_samba3upgrade(Command): """Upgrade from Samba3 database to Samba4 AD database. - Specify either samba3 database directory (with --libdir) or + Specify either a directory with all samba3 databases and state files (with --dbdir) or samba3 testparm utility (with --testparm). """ @@ -542,7 +542,7 @@ class cmd_domain_samba3upgrade(Command): } takes_options = [ - Option("--libdir", type="string", metavar="DIR", + Option("--dbdir", type="string", metavar="DIR", help="Path to samba3 database directory"), Option("--testparm", type="string", metavar="PATH", help="Path to samba3 testparm utility from the previous installation. This allows the default paths of the previous installation to be followed"), @@ -555,7 +555,7 @@ class cmd_domain_samba3upgrade(Command): takes_args = ["smbconf"] - def run(self, smbconf=None, targetdir=None, libdir=None, testparm=None, + def run(self, smbconf=None, targetdir=None, dbdir=None, testparm=None, quiet=None, use_xattrs=None, sambaopts=None, versionopts=None): if not os.path.exists(smbconf): @@ -564,15 +564,15 @@ class cmd_domain_samba3upgrade(Command): if testparm and not os.path.exists(testparm): raise CommandError("Testparm utility %s does not exist" % testparm) - if libdir and not os.path.exists(libdir): - raise CommandError("Directory %s does not exist" % libdir) + if dbdir and not os.path.exists(dbdir): + raise CommandError("Directory %s does not exist" % dbdir) - if not libdir and not testparm: - raise CommandError("Please specify either libdir or testparm") + if not dbdir and not testparm: + raise CommandError("Please specify either dbdir or testparm") - if libdir and testparm: - self.outf.write("warning: both libdir and testparm specified, ignoring libdir.\n") - libdir = None + if dbdir and testparm: + self.outf.write("warning: both dbdir and testparm specified, ignoring dbdir.\n") + dbdir = None logger = self.get_logger() if quiet: @@ -605,12 +605,12 @@ class cmd_domain_samba3upgrade(Command): "If you intend to use this provision in production, rerun the script as root on a system supporting xattrs.") tmpfile.close() - # Set correct default values from libdir or testparm + # Set correct default values from dbdir or testparm paths = {} - if libdir: - paths["state directory"] = libdir - paths["private dir"] = libdir - paths["lock directory"] = libdir + if dbdir: + paths["state directory"] = dbdir + paths["private dir"] = dbdir + paths["lock directory"] = dbdir else: paths["state directory"] = get_testparm_var(testparm, smbconf, "state directory") paths["private dir"] = get_testparm_var(testparm, smbconf, "private dir") diff --git a/source4/scripting/python/samba/upgrade.py b/source4/scripting/python/samba/upgrade.py index 2a2c17ec2c4..67dd333b9e1 100644 --- a/source4/scripting/python/samba/upgrade.py +++ b/source4/scripting/python/samba/upgrade.py @@ -473,7 +473,7 @@ def upgrade_from_samba3(samba3, logger, targetdir, session_info=None, useeadb=Fa try: secrets_db = samba3.get_secrets_db() except IOError, e: - raise ProvisioningError("Could not open '%s', the Samba3 secrets database: %s. Perhaps you specified the incorrect smb.conf, --testparm or --libdir option?" % (samba3.privatedir_path("secrets.tdb"), str(e))) + raise ProvisioningError("Could not open '%s', the Samba3 secrets database: %s. Perhaps you specified the incorrect smb.conf, --testparm or --dbdir option?" % (samba3.privatedir_path("secrets.tdb"), str(e))) if not domainname: domainname = secrets_db.domains()[0] diff --git a/source4/setup/tests/blackbox_s3upgrade.sh b/source4/setup/tests/blackbox_s3upgrade.sh index 2b90573fce0..f93b43f6b91 100755 --- a/source4/setup/tests/blackbox_s3upgrade.sh +++ b/source4/setup/tests/blackbox_s3upgrade.sh @@ -38,7 +38,7 @@ cat - > $PREFIX/samba3-upgrade/samba3/smb1.conf < $PREFIX/samba3-upgrade/samba3/smb2.conf < $PREFIX/samba3-upgrade/samba3/smb2.conf <