search

jmbannon / ytdl-sub

Lightweight tool to automate downloading and metadata generation with yt-dlp
https://ytdl-sub.readthedocs.io
GNU General Public License v3.0
1.69k stars 65 forks source link

Ffmpeg using wrong extension for writing video metadata #1021

Closed eheupel closed 5 days ago

eheupel commented 2 months ago

Getting consistent processing error on one video, looks like coming from ffmpeg throwing a non-sero exit code 254. Attaching log file for that as well as config and sunbscription yaml files.

2024-07-12-141139.tom_mason_sculpts.error.log subscriptions.yam.txt config.yaml.txt

jmbannon commented 2 months ago 
[ytdl-sub:yt-dlp] [Merger] Merging formats into ".ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.mp4"
[ytdl-sub:yt-dlp] Deleting original file .ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.f140.m4a (pass -k to keep)
[ytdl-sub:yt-dlp] Deleting original file .ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.f136.mp4 (pass -k to keep)
[ytdl-sub:yt-dlp] [Metadata] There isn't any metadata to add
[ytdl-sub:ffmpeg] Running /usr/bin/ffmpeg -y -bitexact -i .ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.webp .ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.jpg
[ytdl-sub:ffmpeg] Running /usr/bin/ffmpeg -y -bitexact -i /tmp/tmpmlqow37p /tmp/tmpmlqow37p.out.jpg
[ytdl-sub:ffmpeg] Running /usr/bin/ffmpeg -y -bitexact -i /tmp/tmpzg1grd15 /tmp/tmpzg1grd15.out.jpg
[ytdl-sub:ffmpeg] Running /usr/bin/ffmpeg -i .ytdl-sub-working-directory/Tom Mason Sculpts/bQKeD41qFdg.mkv -map 0 -dn -metadata show=Tom Mason Sculpts -metadata genre=MiniPainting -metadata episode_id=21501 -metadata title=100 Days of Making Minis - Day 9 -metadata synopsis=https://www.youtube.com/watch?v=bQKeD41qFdg

Seems like it's trying to run ffmpeg on mkv, but it's actually an mp4. I will try to repro and fix

eheupel commented 2 months ago

Much thanks!

jmbannon commented 5 days ago

Should be fixed by the above PR, please reopen if that is not the case.