]> git.bitcoin.ninja Git - youtube-dl/commitdiff
[xattr] Add version detection for python-pyxattr
authorYen Chi Hsuan <yan12125@gmail.com>
Thu, 23 Apr 2015 05:49:06 +0000 (13:49 +0800)
committerYen Chi Hsuan <yan12125@gmail.com>
Thu, 23 Apr 2015 05:50:44 +0000 (13:50 +0800)
For more information, see #5498 and changes to convertObj() in
iustin/pyxattr@cc84e466f63906d32ec1bf4a4fcae6a7bce9a4c8

youtube_dl/postprocessor/xattrpp.py

index 0cba99fc3168a630b97a8b923292e8a6b604d1c6..b74adff43d884100ac1fafe197464db489dd7e34 100644 (file)
@@ -11,6 +11,7 @@ from ..compat import (
 from ..utils import (
     check_executable,
     hyphenate_date,
+    version_tuple,
 )
 
 
@@ -36,6 +37,19 @@ class XAttrMetadataPP(PostProcessor):
             # try the pyxattr module...
             import xattr
 
+            # Unicode arguments are not supported in python-pyxattr until
+            # version 0.5.0
+            # See https://github.com/rg3/youtube-dl/issues/5498
+            pyxattr_required_version = '0.5.0'
+            if version_tuple(xattr.__version__) < version_tuple(pyxattr_required_version):
+                self._downloader.report_warning(
+                    'python-pyxattr is detected but is too old. '
+                    'yourube-dl requires %s or above while your version is %s. '
+                    'Falling back to other xattr implementations' % (
+                        pyxattr_required_version, xattr.__version__))
+
+                raise ImportError
+
             def write_xattr(path, key, value):
                 return xattr.setxattr(path, key, value)