Merge pull request #128 from xenserver/private/andrewcoop/rm-simplejson

Remove the use of simplejson

pyproject.toml

@@ -68,7 +68,6 @@ mypy = [
     "mypy-extensions",
     "typing_extensions",
     "types-mock",
-    "types-simplejson",
     "types-six",
     "types-toml",
 ]

xcp/net/ifrename/dynamic.py

@@ -31,19 +31,11 @@
 from __future__ import unicode_literals
 
 from os.path import exists as pathexists
+import json
 
 __version__ = "1.0.0"
 __author__  = "Andrew Cooper"
 
-try:
-    import json
-except ImportError:
-    try:
-        import simplejson as json  # type: ignore[no-redef]  # pragma: no cover
-    # The installer has no json.  In the meantime, there is a workaround
-    except ImportError:
-        pass
-
 
 from xcp.compat import open_with_codec_handling
 from xcp.logger import LOG
@@ -126,10 +118,6 @@ def load_and_parse(self):
         except ValueError:
             LOG.warning("Dynamic rules appear to be corrupt")
             return False
-        # The installer has no json.
-        except NameError:
-            LOG.warning("Module json not available.  Cant parse dynamic rules.")
-            return False
 
         if "lastboot" in info:
             for entry in info["lastboot"]:
@@ -263,15 +251,8 @@ def validate(entry):
         lastboot = [x for x in self.lastboot if validate(x)]
         old = [x for x in self.old if validate(x)]
 
-        try:
-            res += json.dumps({"lastboot": lastboot, "old": old},
-                              indent=4, sort_keys=True)
-            # Installer has no json.  This will do in the meantime
-        except NameError:
-            res += ('{"lastboot":%s,"old":%s}'
-                    % ( ("%s" % (lastboot,)).replace("'", '"'),
-                        ("%s" % (old,)).replace("'", '"'))
-                    )
+        res += json.dumps({"lastboot": lastboot, "old": old},
+                          indent=4, sort_keys=True)
 
         return res