about summary refs log tree commit diff
path: root/youtube_dl/downloader/hls.py
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2016-02-06 23:24:22 +0600
committerSergey M․ <dstftw@gmail.com>2016-02-06 23:24:22 +0600
commit56401e1e5f53e126bf124699b2f62bb670078fd3 (patch)
tree6865b90ee09bfb839bb901d040acbe5198e43b1c /youtube_dl/downloader/hls.py
parent860db2d508f2a18c1b48adb9b1eddc753721bd47 (diff)
downloadyoutube-dl-56401e1e5f53e126bf124699b2f62bb670078fd3.tar.gz
youtube-dl-56401e1e5f53e126bf124699b2f62bb670078fd3.tar.xz
youtube-dl-56401e1e5f53e126bf124699b2f62bb670078fd3.zip
[downloader/hls] Do not send 'q' to ffmpeg on Windows (Closes #8300)
Diffstat (limited to 'youtube_dl/downloader/hls.py')
-rw-r--r--youtube_dl/downloader/hls.py7
1 files changed, 5 insertions, 2 deletions
diff --git a/youtube_dl/downloader/hls.py b/youtube_dl/downloader/hls.py
index cb34dc4ab..2a775bf00 100644
--- a/youtube_dl/downloader/hls.py
+++ b/youtube_dl/downloader/hls.py
@@ -3,6 +3,7 @@ from __future__ import unicode_literals
 import os
 import re
 import subprocess
+import sys
 
 from .common import FileDownloader
 from .fragment import FragmentFD
@@ -57,8 +58,10 @@ class HlsFD(FileDownloader):
             # subprocces.run would send the SIGKILL signal to ffmpeg and the
             # mp4 file couldn't be played, but if we ask ffmpeg to quit it
             # produces a file that is playable (this is mostly useful for live
-            # streams)
-            proc.communicate(b'q')
+            # streams). Note that Windows is not affected and produces playable
+            # files (see https://github.com/rg3/youtube-dl/issues/8300).
+            if sys.platform != 'win32':
+                proc.communicate(b'q')
             raise
         if retval == 0:
             fsize = os.path.getsize(encodeFilename(tmpfilename))