From 92120217eb27d820bfba0f6a50837f111cbf80ef Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jaime=20Marqui=CC=81nez=20Ferra=CC=81ndiz?= Date: Tue, 18 Nov 2014 23:28:42 +0100 Subject: [PATCH] [cache] Fix writing to paths with unicode characters * Use "compat_getenv" * "write_json_file" now expects the filename to be a string --- youtube_dl/YoutubeDL.py | 2 +- youtube_dl/cache.py | 4 ++-- youtube_dl/utils.py | 1 + 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/youtube_dl/YoutubeDL.py b/youtube_dl/YoutubeDL.py index 5c875b497..94c50903c 100755 --- a/youtube_dl/YoutubeDL.py +++ b/youtube_dl/YoutubeDL.py @@ -1001,7 +1001,7 @@ def process_info(self, info_dict): else: self.to_screen('[info] Writing video description metadata as JSON to: ' + infofn) try: - write_json_file(info_dict, encodeFilename(infofn)) + write_json_file(info_dict, infofn) except (OSError, IOError): self.report_error('Cannot write metadata to JSON file ' + infofn) return diff --git a/youtube_dl/cache.py b/youtube_dl/cache.py index 2d9b426cb..5fe839eb1 100644 --- a/youtube_dl/cache.py +++ b/youtube_dl/cache.py @@ -8,7 +8,7 @@ import shutil import traceback -from .compat import compat_expanduser +from .compat import compat_expanduser, compat_getenv from .utils import write_json_file @@ -19,7 +19,7 @@ def __init__(self, ydl): def _get_root_dir(self): res = self._ydl.params.get('cachedir') if res is None: - cache_root = os.environ.get('XDG_CACHE_HOME', '~/.cache') + cache_root = compat_getenv('XDG_CACHE_HOME', '~/.cache') res = os.path.join(cache_root, 'youtube-dl') return compat_expanduser(res) diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index 50e515a04..94b496dd0 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -73,6 +73,7 @@ def preferredencoding(): def write_json_file(obj, fn): """ Encode obj as JSON and write it to fn, atomically """ + fn = encodeFilename(fn) if sys.version_info < (3, 0): encoding = get_filesystem_encoding() # os.path.basename returns a bytes object, but NamedTemporaryFile