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

mid3v2 produces file that mutagen can't read again #634

Closed
xeals opened this issue Nov 9, 2023 · 1 comment
Closed

mid3v2 produces file that mutagen can't read again #634

xeals opened this issue Nov 9, 2023 · 1 comment

Comments

@xeals
Copy link

xeals commented Nov 9, 2023

Encountered with at least opus and aac (m4a container) audio. Reproducible with:

$ yt-dlp --extract-audio --audio-format opus --output file.opus <any YouTube URL>
$ mid3v2 -a art -A alb -T 1 file.opus
$ mutagen-inspect file.opus
-- file.opus
can't sync to MPEG frame
$ mid3v2 -l file.opus
IDv2 tag into for file.opus
TALB=alb
TPE1=art
TRCK=1

mutagen-inspect is used as an example of other programs that use Mutagen as a library, which this also affects.

@phw
Copy link
Collaborator

phw commented Nov 9, 2023

Please see #604

That's because mid3v2 is a utility to work with ID3 tags and is adding ID3 tags to the files. It should not be used on non-ID3 formats like opus.

Ideally the command should refuse to modify the Opus file, but currently it just adds the tags.

@phw phw closed this as completed Nov 9, 2023
@phw phw added the duplicate label Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants