[core] deprecation for ddtrace.util, ddtrace.contrib.util and safe_patch; they will note be used anymore

* `ddtrace.util` is replaced with `ddtrace.utils` package
* `ddtrace.contrib.util` is replaced with `ddtrace.utils.importlib`
* `safe_patch` will be removed because not used anymore

Author: Emanuele Palazzetti

ddtrace/contrib/util.py

@@ -1,33 +1,16 @@
-from importlib import import_module
-
-
-class require_modules(object):
-    """
-    Context manager to check the availability of required modules.
-    """
-    def __init__(self, modules):
-        self._missing_modules = []
-        for module in modules:
-            try:
-                import_module(module)
-            except ImportError:
-                self._missing_modules.append(module)
-
-    def __enter__(self):
-        return self._missing_modules
-
-    def __exit__(self, exc_type, exc_value, traceback):
-        return False
-
-
-def func_name(f):
-    """
-    Return a human readable version of the function's name.
-    """
-    if hasattr(f, '__module__'):
-        return "%s.%s" % (f.__module__, getattr(f, '__name__', f.__class__.__name__))
-    return getattr(f, '__name__', f.__class__.__name__)
-
-
-def module_name(instance):
-    return instance.__class__.__module__.split('.')[0]
+# [Backward compatibility]: keep importing modules functions
+from ..utils.deprecation import deprecation
+from ..utils.importlib import require_modules, func_name, module_name
+
+
+deprecation(
+    name='ddtrace.contrib.util',
+    message='Use `ddtrace.utils.importlib` module instead',
+    version='1.0.0',
+)
+
+__all__ = [
+    'require_modules',
+    'func_name',
+    'module_name',
+]

ddtrace/util.py

@@ -1,9 +1,15 @@
 # [Backward compatibility]: keep importing modules functions
-from .utils.deprecation import deprecated
+from .utils.deprecation import deprecated, deprecation
 from .utils.formats import asbool, deep_getattr, get_env
 from .utils.wrappers import safe_patch, unwrap
 
 
+deprecation(
+    name='ddtrace.util',
+    message='Use `ddtrace.utils` package instead',
+    version='1.0.0',
+)
+
 __all__ = [
     'deprecated',
     'asbool',

ddtrace/utils/importlib.py

@@ -0,0 +1,30 @@
+from importlib import import_module
+
+
+class require_modules(object):
+    """Context manager to check the availability of required modules."""
+    def __init__(self, modules):
+        self._missing_modules = []
+        for module in modules:
+            try:
+                import_module(module)
+            except ImportError:
+                self._missing_modules.append(module)
+
+    def __enter__(self):
+        return self._missing_modules
+
+    def __exit__(self, exc_type, exc_value, traceback):
+        return False
+
+
+def func_name(f):
+    """Return a human readable version of the function's name."""
+    if hasattr(f, '__module__'):
+        return "%s.%s" % (f.__module__, getattr(f, '__name__', f.__class__.__name__))
+    return getattr(f, '__name__', f.__class__.__name__)
+
+
+def module_name(instance):
+    """Return the instance module name."""
+    return instance.__class__.__module__.split('.')[0]

ddtrace/utils/wrappers.py

@@ -1,13 +1,16 @@
 import wrapt
 import inspect
 
+from .deprecation import deprecated
+
 
 def unwrap(obj, attr):
     f = getattr(obj, attr, None)
     if f and isinstance(f, wrapt.ObjectProxy) and hasattr(f, '__wrapped__'):
         setattr(obj, attr, f.__wrapped__)
 
 
+@deprecated('`wrapt` library is used instead', version='1.0.0')
 def safe_patch(patchable, key, patch_func, service, meta, tracer):
     """ takes patch_func (signature: takes the orig_method that is
     wrapped in the monkey patch == UNBOUND + service and meta) and