diff options
author | Yen Chi Hsuan <yan12125@gmail.com> | 2016-10-22 13:10:27 +0800 |
---|---|---|
committer | Yen Chi Hsuan <yan12125@gmail.com> | 2016-10-22 13:10:27 +0800 |
commit | e034cbc581c79b14c6337adf51bf4a99f726302d (patch) | |
tree | e766cf2817c4a5fed4002fe674a7c08acce9b925 | |
parent | 69c2d42bd730b4ea07fe5ba9015049423b71c8a9 (diff) | |
parent | 5378f8ce0d59d0a948d2597b175b2b1cff3e8bb2 (diff) |
Merge branch 'johnhawkinson-stdin2'
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | youtube_dl/utils.py | 4 |
2 files changed, 10 insertions, 0 deletions
@@ -1,3 +1,9 @@ +version <unreleased> + +Core +* Running youtube-dl in the background is fixed (#10996, #10706, #955) + + version 2016.10.21.1 Extractors diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index 28941673f..2770c5f1c 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -1818,8 +1818,12 @@ def get_exe_version(exe, args=['--version'], """ Returns the version of the specified executable, or False if the executable is not present """ try: + # STDIN should be redirected too. On UNIX-like systems, ffmpeg triggers + # SIGTTOU if youtube-dl is run in the background. + # See https://github.com/rg3/youtube-dl/issues/955#issuecomment-209789656 out, _ = subprocess.Popen( [encodeArgument(exe)] + args, + stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.STDOUT).communicate() except OSError: return False |