diff options
author | Sergey M․ <dstftw@gmail.com> | 2015-04-26 04:33:43 +0600 |
---|---|---|
committer | Sergey M․ <dstftw@gmail.com> | 2015-04-26 04:33:43 +0600 |
commit | 74f8654a53ed56dd813ace704151ea575963cabb (patch) | |
tree | 2b9dd53ca471e8adbd5893e83d08f95d26973fc4 /youtube_dl/downloader | |
parent | 9e105a858c2aa0089764326cc3fca808c51a9ffd (diff) |
[downloader/external] Use encodeArgument
Diffstat (limited to 'youtube_dl/downloader')
-rw-r--r-- | youtube_dl/downloader/external.py | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/youtube_dl/downloader/external.py b/youtube_dl/downloader/external.py index 1673b2382..7ca2d3143 100644 --- a/youtube_dl/downloader/external.py +++ b/youtube_dl/downloader/external.py @@ -2,11 +2,11 @@ from __future__ import unicode_literals import os.path import subprocess -import sys from .common import FileDownloader from ..utils import ( encodeFilename, + encodeArgument, ) @@ -60,17 +60,9 @@ class ExternalFD(FileDownloader): def _call_downloader(self, tmpfilename, info_dict): """ Either overwrite this or implement _make_cmd """ - cmd = self._make_cmd(tmpfilename, info_dict) - - if sys.platform == 'win32' and sys.version_info < (3, 0): - # Windows subprocess module does not actually support Unicode - # on Python 2.x - # See http://stackoverflow.com/a/9951851/35070 - subprocess_encoding = sys.getfilesystemencoding() - cmd = [a.encode(subprocess_encoding, 'ignore') for a in cmd] - else: - subprocess_encoding = None - self._debug_cmd(cmd, subprocess_encoding) + cmd = [encodeArgument(a) for a in self._make_cmd(tmpfilename, info_dict)] + + self._debug_cmd(cmd) p = subprocess.Popen( cmd, stderr=subprocess.PIPE) |