Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ffmpeg + ogg vorbis: header remain warning #617

Open
rmeissn opened this issue Jun 19, 2023 · 1 comment
Open

ffmpeg + ogg vorbis: header remain warning #617

rmeissn opened this issue Jun 19, 2023 · 1 comment

Comments

@rmeissn
Copy link

rmeissn commented Jun 19, 2023

I've used Ear Tag to batch tag some ogg vorbis files. Ear Tag uses mutagen under the hood.

After tagging, ffmpegs ffprobe showed [ogg @ 0x556109c47000] 5929 bytes of comment header remain as a warning about the file, which wasn't there before tagging it. Over several files, the address and byte numbers change, but the warning remains.

The dev of Ear Tag requested to post the issue here. See https://gitlab.gnome.org/World/eartag/-/issues/83#note_1772588

@lazka
Copy link
Member

lazka commented Sep 15, 2023

This is due to tag padding: https://mutagen.readthedocs.io/en/latest/user/padding.html

packets[0] = vcomment_data + b"\x00" * new_padding

If ffmpeg has a good reason why this should print a warning, or if there is a better way to handle tag padding in oggvorbis, I'm open to ideas.

@lazka lazka changed the title header remain warning ffmpeg + ogg vorbis: header remain warning Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants