python/samba: changes to make samba.tests.samba_tool.join run under py3
authorNoel Power <noel.power@suse.com>
Tue, 17 Apr 2018 15:48:03 +0000 (16:48 +0100)
committerAndrew Bartlett <abartlet@samba.org>
Mon, 3 Sep 2018 01:22:21 +0000 (03:22 +0200)
Signed-off-by: Noel Power <noel.power@suse.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
python/samba/join.py
python/samba/netcmd/domain_backup.py

index 9cab604ec4a26f760d52ab72813af1d60e12a2be..e7963eeaa1e1ddcb65831abb11ad4b7f8be2e1b2 100644 (file)
@@ -354,21 +354,21 @@ class DCJoinContext(object):
 
     def get_dnsHostName(ctx):
         res = ctx.samdb.search(base="", scope=ldb.SCOPE_BASE, attrs=["dnsHostName"])
-        return res[0]["dnsHostName"][0]
+        return str(res[0]["dnsHostName"][0])
 
     def get_domain_name(ctx):
         '''get netbios name of the domain from the partitions record'''
         partitions_dn = ctx.samdb.get_partitions_dn()
         res = ctx.samdb.search(base=partitions_dn, scope=ldb.SCOPE_ONELEVEL, attrs=["nETBIOSName"],
                                expression='ncName=%s' % ldb.binary_encode(str(ctx.samdb.get_default_basedn())))
-        return res[0]["nETBIOSName"][0]
+        return str(res[0]["nETBIOSName"][0])
 
     def get_forest_domain_name(ctx):
         '''get netbios name of the domain from the partitions record'''
         partitions_dn = ctx.samdb.get_partitions_dn()
         res = ctx.samdb.search(base=partitions_dn, scope=ldb.SCOPE_ONELEVEL, attrs=["nETBIOSName"],
                                expression='ncName=%s' % ldb.binary_encode(str(ctx.samdb.get_root_basedn())))
-        return res[0]["nETBIOSName"][0]
+        return str(res[0]["nETBIOSName"][0])
 
     def get_parent_partition_dn(ctx):
         '''get the parent domain partition DN from parent DNS name'''
index c2f056e6ba3879ee7c7119f7301c8539e5fed62f..de53c6e5113c79642d9c67a5dc758c102fefde73 100644 (file)
@@ -891,7 +891,7 @@ class cmd_domain_backup_offline(samba.netcmd.Command):
         # If more than one directory is a parent of this path, then at least
         # one configured path is a subdir of another. Use closest match.
         if len(matching_dirs) > 1:
-            arc_path, fs_path = max(matching_dirs, key=lambda (_, p): len(p))
+            arc_path, fs_path = max(matching_dirs, key=lambda p: len(p[1]))
         arc_path += path[len(fs_path):]
 
         return arc_path