Commit Graph

13 Commits (9a269547f2268822724018330c8a939d3bdc2db4)

Author SHA1 Message Date
Andrew Udvare b5cbe3d652 [postprocessor/embedthumbnail] Skip embedding when there aren't any thumbnails
Yen Chi Hsuan 8e2915d70b
Revert "[postprocessor/embedthumbnail] Allow mkv to embed thumbnails"
This reverts commit 7360db05b4.

This commit was added as an attempt to fix . Unfortunately, the fix
is completely wrong. As reported on , 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.
Déstin Reed dcdb292fdd
Unify coding cookie
Yen Chi Hsuan 7360db05b4 [postprocessor/embedthumbnail] Allow mkv to embed thumbnails
Fixes 
Yen Chi Hsuan c33a8639a7 [postprocessor/embedthumbnail] Skipping if thumbnail file missing
Fixes 
pulpe d6aa68ce75 [postprocessor/embedthumbnail] embed mp4 too (fixes )
Yen Chi Hsuan 2cc6d13547 [postprocessor/embedthumbnail] Encode arguments in calling AtomicParsley
Yen Chi Hsuan bb8ca1d112 [postprocessor/embedthumbnail] Use run_ffmpeg_multiple_files
Yen Chi Hsuan 8e59539752 [postprocessor/embedthumbnail] Use thumbnails downloaded by YoutubeDL
pulpe 2a09c1b8ab [postprocessor/embedthumbnail] Fix mp3 embedding with avconv (fixes )
Jaime Marquínez Ferrándiz 92995e6265 [postprocessor/embedthumbnail] Style fix
pulpe 31fd9c7601 [embedthumbnail] use FFmpegPostProcessor for mp3
pulpe ddbed36455 [embedthumbnail] Add support for mp3 cover embedding