summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2015-04-26 04:33:43 +0600
committerSergey M․ <dstftw@gmail.com>2015-04-26 04:33:43 +0600
commit74f8654a53ed56dd813ace704151ea575963cabb (patch)
tree2b9dd53ca471e8adbd5893e83d08f95d26973fc4
parent9e105a858c2aa0089764326cc3fca808c51a9ffd (diff)
downloadyoutube-dl-74f8654a53ed56dd813ace704151ea575963cabb.tar.gz
youtube-dl-74f8654a53ed56dd813ace704151ea575963cabb.tar.xz
youtube-dl-74f8654a53ed56dd813ace704151ea575963cabb.zip
[downloader/external] Use encodeArgument
-rw-r--r--youtube_dl/downloader/external.py16
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)