Skip to content

Add id3v2 mappings for MusicBrainz and Vorbis tags#60

Open
dajhorn wants to merge 2 commits intoZivaVatra:masterfrom
dajhorn:id3v2
Open

Add id3v2 mappings for MusicBrainz and Vorbis tags#60
dajhorn wants to merge 2 commits intoZivaVatra:masterfrom
dajhorn:id3v2

Conversation

@dajhorn
Copy link

@dajhorn dajhorn commented Feb 23, 2023

Preserve FLAC metadata (metaflac comments) from music libraries that are
managed by Picard or other compatible management tools.

https://picard-docs.musicbrainz.org/en/appendices/tag_mapping.html

Resolve this build warning:

	/usr/local/lib/python3.10/dist-packages/flac2all-5.5-py3.10.egg/flac2all_pkg/core.py:248: SyntaxWarning: "is" with a literal. Did you mean "=="?
	  if opts['nodirs'] is "d":
	/usr/local/lib/python3.10/dist-packages/flac2all-5.5-py3.10.egg/flac2all_pkg/core.py:253: SyntaxWarning: "is" with a literal. Did you mean "=="?
	  elif opts['nodirs'] is "m":
Preserve FLAC metadata (metaflac comments) from music libraries that are
managed by Picard or other compatible management tools.

https://picard-docs.musicbrainz.org/en/appendices/tag_mapping.html
@ZivaVatra
Copy link
Owner

Has this been tested for regression from music libraries not managed by the tools you mentioned?

@dajhorn
Copy link
Author

dajhorn commented May 30, 2023

This is not tested against non-MetaBrainz input short of preserving existing behavior for id3v1 and CDDB tags.

Output was tested with Foobar2000, Clementine, Windows Media Player, and the CD-R player in my car.

I noticed one glitch: Picard will write out a UFID tag to an MP3 file even if it already has a corresponding TXXX tag. Thus, MP3 files that are updated by Picard after intitial FLAC conversion will have the Track ID in both the UFID tag and a TXXX tag.

@ZivaVatra ZivaVatra self-assigned this May 14, 2024
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

Successfully merging this pull request may close these issues.

2 participants