youtube-dl

commit: e205db3bcd7f7a9b0beaa13a4774bdcb26091e2f
parent 31d4a6e2121ab5954aec5cfb48d6b1f1cbffefbd
Author: Jaime Marquínez Ferrándiz <jaime.marquinez.ferrandiz@gmail.com>
Date:   Fri, 16 Jan 2015 13:29:01 +0100

FFmpegEmbedSubtitlePP: don't fail if the video doesn't have an audio stream (fixes #4718)

Instead of specifying which streams ffmpeg must copy, we tell it to copy all.

Diffstat:
M
youtube_dl/postprocessor/ffmpeg.py
8
+++++++-

1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/youtube_dl/postprocessor/ffmpeg.py b/youtube_dl/postprocessor/ffmpeg.py
@@ -475,7 +475,13 @@ class FFmpegEmbedSubtitlePP(FFmpegPostProcessor):
         filename = information['filepath']
         input_files = [filename] + [subtitles_filename(filename, lang, self._subformat) for lang in sub_langs]
 
-        opts = ['-map', '0:0', '-map', '0:1', '-c:v', 'copy', '-c:a', 'copy']
+        opts = [
+            '-map', '0',
+            '-c', 'copy',
+            # Don't copy the existing subtitles, we may be running the
+            # postprocessor a second time
+            '-map', '-0:s',
+        ]
         for (i, lang) in enumerate(sub_langs):
             opts.extend(['-map', '%d:0' % (i + 1), '-c:s:%d' % i, 'mov_text'])
             lang_code = self._conver_lang_code(lang)