From: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Date: Sat, 4 Jul 2020 01:47:44 +0000 (+1200)
Subject: python compat: remove string_types
X-Git-Tag: talloc-2.3.2~880
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=9cc65a552b0270db0ea08509ee6e9a3d5b9a1f10;p=thirdparty%2Fsamba.git

python compat: remove string_types

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Noel Power <npower@samba.org>
---

diff --git a/python/samba/__init__.py b/python/samba/__init__.py
index 6272b3fb390..360918c49ad 100644
--- a/python/samba/__init__.py
+++ b/python/samba/__init__.py
@@ -29,7 +29,6 @@ import ldb
 import samba.param
 from samba import _glue
 from samba._ldb import Ldb as _Ldb
-from samba.compat import string_types
 
 
 def source_tree_topdir():
@@ -250,8 +249,8 @@ def substitute_var(text, values):
     """
 
     for (name, value) in values.items():
-        assert isinstance(name, string_types), "%r is not a string" % name
-        assert isinstance(value, string_types), "Value %r for %s is not a string" % (value, name)
+        assert isinstance(name, str), "%r is not a string" % name
+        assert isinstance(value, str), "Value %r for %s is not a string" % (value, name)
         text = text.replace("${%s}" % name, value)
 
     return text
diff --git a/python/samba/compat.py b/python/samba/compat.py
index 9ef60045bd3..01849e0eef5 100644
--- a/python/samba/compat.py
+++ b/python/samba/compat.py
@@ -72,7 +72,6 @@ if PY3:
     from functools import cmp_to_key as cmp_to_key_fn
 
     # compat types
-    string_types = str
     text_type = str
     binary_type = bytes
 
diff --git a/python/samba/ms_schema.py b/python/samba/ms_schema.py
index 4946636cbd4..4b5c4a5df83 100644
--- a/python/samba/ms_schema.py
+++ b/python/samba/ms_schema.py
@@ -21,7 +21,6 @@ from __future__ import print_function
 import re
 import base64
 import uuid
-from samba.compat import string_types
 
 bitFields = {}
 
@@ -178,7 +177,7 @@ def fix_dn(dn):
 
 def __convert_bitfield(key, value):
     """Evaluate the OR expression in 'value'"""
-    assert(isinstance(value, string_types))
+    assert(isinstance(value, str))
 
     value = value.replace("\n ", "")
     value = value.replace(" ", "")
@@ -201,7 +200,7 @@ def __write_ldif_one(entry):
     out = []
 
     for l in entry:
-        if isinstance(l[1], string_types):
+        if isinstance(l[1], str):
             vl = [l[1]]
         else:
             vl = l[1]
@@ -254,7 +253,7 @@ def __transform_entry(entry, objectClass):
                 l[1] = oMObjectClassBER[l[1].strip()]
                 l[2] = True
 
-        if isinstance(l[1], string_types):
+        if isinstance(l[1], str):
             l[1] = fix_dn(l[1])
 
         if key == 'dn':
diff --git a/python/samba/provision/__init__.py b/python/samba/provision/__init__.py
index 3f176e00ba4..4ac89ef406f 100644
--- a/python/samba/provision/__init__.py
+++ b/python/samba/provision/__init__.py
@@ -27,7 +27,6 @@
 __docformat__ = "restructuredText"
 
 from urllib.parse import quote as urllib_quote
-from samba.compat import string_types
 from samba.compat import binary_type
 from base64 import b64encode
 import errno
@@ -1606,7 +1605,7 @@ def fill_samdb(samdb, lp, names, logger, policyguid,
         ntds_dn = "CN=NTDS Settings,%s" % names.serverdn
         names.ntdsguid = samdb.searchone(basedn=ntds_dn,
                                          attribute="objectGUID", expression="", scope=ldb.SCOPE_BASE).decode('utf8')
-        assert isinstance(names.ntdsguid, string_types)
+        assert isinstance(names.ntdsguid, str)
 
     return samdb
 
@@ -2014,7 +2013,7 @@ def provision_fill(samdb, secrets_ldb, logger, names, paths,
 
         domainguid = samdb.searchone(basedn=samdb.get_default_basedn(),
                                      attribute="objectGUID").decode('utf8')
-        assert isinstance(domainguid, string_types)
+        assert isinstance(domainguid, str)
 
     lastProvisionUSNs = get_last_provision_usn(samdb)
     maxUSN = get_max_usn(samdb, str(names.rootdn))
diff --git a/python/samba/tests/__init__.py b/python/samba/tests/__init__.py
index 768231384d8..3ff11497ef6 100644
--- a/python/samba/tests/__init__.py
+++ b/python/samba/tests/__init__.py
@@ -35,7 +35,6 @@ from enum import IntEnum, unique
 import samba.auth
 import samba.dcerpc.base
 from samba.compat import text_type
-from samba.compat import string_types
 from random import randint
 from random import SystemRandom
 from contextlib import contextmanager
@@ -92,7 +91,7 @@ class TestCase(unittest.TestCase):
     def hexdump(self, src):
         N = 0
         result = ''
-        is_string = isinstance(src, string_types)
+        is_string = isinstance(src, str)
         while src:
             ll = src[:8]
             lr = src[8:16]