]> git.bitcoin.ninja Git - youtube-dl/commitdiff
Revert "[postprocessor/embedthumbnail] Allow mkv to embed thumbnails"
authorYen Chi Hsuan <yan12125@gmail.com>
Thu, 20 Oct 2016 07:07:19 +0000 (15:07 +0800)
committerYen Chi Hsuan <yan12125@gmail.com>
Thu, 20 Oct 2016 07:07:19 +0000 (15:07 +0800)
This reverts commit 7360db05b43741c2dfa1fd024e9c2f013ed97c9e.

This commit was added as an attempt to fix #6046. Unfortunately, the fix
is completely wrong. As reported on #10359, embedded thumbnails are not
displayed in VLC, and Se7en on IRC reports that the embedded thumbnail
misleads mpv as well.

The correct way is using -attachment of ffmpeg, while the current
run_ffmpeg_multiple_files API can't handle it cleanly.

youtube_dl/postprocessor/embedthumbnail.py

index 2e4789eb220b1069f51c5e68a9b3ff76b7514c6d..e606a58de886533fb5239b9bb958fbff9606a4ee 100644 (file)
@@ -40,7 +40,7 @@ class EmbedThumbnailPP(FFmpegPostProcessor):
                 'Skipping embedding the thumbnail because the file is missing.')
             return [], info
 
-        if info['ext'] in ('mp3', 'mkv'):
+        if info['ext'] == 'mp3':
             options = [
                 '-c', 'copy', '-map', '0', '-map', '1',
                 '-metadata:s:v', 'title="Album cover"', '-metadata:s:v', 'comment="Cover (Front)"']