summary refs log tree commit diff
diff options
context:
space:
mode:
authorJaime Marquínez Ferrándiz <jaime.marquinez.ferrandiz@gmail.com>2015-01-04 14:02:17 +0100
committerJaime Marquínez Ferrándiz <jaime.marquinez.ferrandiz@gmail.com>2015-01-04 14:02:17 +0100
commitbc3e582fe457f9239dc3a3386cbfd0e7db167404 (patch)
tree5b6c587544d854e7031fb0036b80e4e6580d2740
parent63948fc62c7f0bfcfe7b2ce102ab6e4e87de558c (diff)
downloadyoutube-dl-bc3e582fe457f9239dc3a3386cbfd0e7db167404.tar.gz
youtube-dl-bc3e582fe457f9239dc3a3386cbfd0e7db167404.tar.xz
youtube-dl-bc3e582fe457f9239dc3a3386cbfd0e7db167404.zip
Don't use '-shortest' option for merging formats (closes #4220, closes #4580)
With avconv and older versions of ffmpeg the video is partially copied.
The duration difference between the audio and the video seem to be really small, so it's probably not noticeable.
-rw-r--r--youtube_dl/postprocessor/ffmpeg.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/youtube_dl/postprocessor/ffmpeg.py b/youtube_dl/postprocessor/ffmpeg.py
index 048525efc..473536dcc 100644
--- a/youtube_dl/postprocessor/ffmpeg.py
+++ b/youtube_dl/postprocessor/ffmpeg.py
@@ -520,7 +520,7 @@ class FFmpegMetadataPP(FFmpegPostProcessor):
 class FFmpegMergerPP(FFmpegPostProcessor):
     def run(self, info):
         filename = info['filepath']
-        args = ['-c', 'copy', '-map', '0:v:0', '-map', '1:a:0', '-shortest']
+        args = ['-c', 'copy', '-map', '0:v:0', '-map', '1:a:0']
         self._downloader.to_screen('[ffmpeg] Merging formats into "%s"' % filename)
         self.run_ffmpeg_multiple_files(info['__files_to_merge'], filename, args)
         return True, info