diff options
| author | Philipp Hagemeister <phihag@phihag.de> | 2014-01-07 06:12:28 +0100 |
|---|---|---|
| committer | Philipp Hagemeister <phihag@phihag.de> | 2014-01-07 06:12:28 +0100 |
| commit | 2a2e2770cc216e3a0d29eb3d164b62bc97938176 (patch) | |
| tree | 051513577fe9bda5ef2e88e488afd3d86f2308d0 | |
| parent | 42cc71e80b9867574234eaa155171545192c0d10 (diff) | |
| download | youtube-dl-2a2e2770cc216e3a0d29eb3d164b62bc97938176.tar.gz youtube-dl-2a2e2770cc216e3a0d29eb3d164b62bc97938176.tar.xz youtube-dl-2a2e2770cc216e3a0d29eb3d164b62bc97938176.zip | |
[xattr] Always output a warning message on errors
| -rw-r--r-- | youtube_dl/postprocessor/xattrpp.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/youtube_dl/postprocessor/xattrpp.py b/youtube_dl/postprocessor/xattrpp.py index 5811d3568..ac2236f35 100644 --- a/youtube_dl/postprocessor/xattrpp.py +++ b/youtube_dl/postprocessor/xattrpp.py @@ -76,9 +76,16 @@ class XAttrMetadataPP(PostProcessor): else: # On Unix, and can't find pyxattr, setfattr, or xattr. if sys.platform.startswith('linux'): - self._downloader.report_error("Couldn't find a tool to set the xattrs. Install either the python 'pyxattr' or 'xattr' modules, or the GNU 'attr' package (which contains the 'setfattr' tool).") - elif sys.platform == 'darwin': - self._downloader.report_error("Couldn't find a tool to set the xattrs. Install either the python 'xattr' module, or the 'xattr' binary.") + self._downloader.report_error( + "Couldn't find a tool to set the xattrs. " + "Install either the python 'pyxattr' or 'xattr' " + "modules, or the GNU 'attr' package " + "(which contains the 'setfattr' tool).") + else: + self._downloader.report_error( + "Couldn't find a tool to set the xattrs. " + "Install either the python 'xattr' module, " + "or the 'xattr' binary.") else: # Write xattrs to NTFS Alternate Data Streams: http://en.wikipedia.org/wiki/NTFS#Alternate_data_streams_.28ADS.29 def write_xattr(path, key, value): |