Show lxml warning only in verbose 3 mode

mar10 · Aug 12, 2018 · f633ba5 · f633ba5
1 parent f9e84c7
commit f633ba5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/wsgidav/server/server_cli.py b/wsgidav/server/server_cli.py
@@ -712,10 +712,10 @@ def run():
             )
         )
 
-    if not use_lxml:  # and config["verbose"] >= 1:
+    if not use_lxml and config["verbose"] >= 3:
         _logger.warn(
-            "Could not import lxml: using xml instead (slower). "
-            "Consider installing lxml https://pypi.python.org/pypi/lxml."
+            "Could not import lxml: using xml instead (up to 10% slower). "
+            "Consider `pip install lxml`(see https://pypi.python.org/pypi/lxml)."
         )
 
     handler(app, config, server)

diff --git a/wsgidav/xml_tools.py b/wsgidav/xml_tools.py
@@ -29,6 +29,7 @@
     use_lxml = True
     _ElementType = etree._Element
 except ImportError:
+    # warnings.warn("Could not import lxml")  # , ImportWarning)
     # Try xml module (Python 2.5 or later) with safe defaults
     from defusedxml import ElementTree as etree