mpv-mpris icon indicating copy to clipboard operation
mpv-mpris copied to clipboard

expose musicbrainz metadata if available

Open tpeacock19 opened this issue 2 years ago • 4 comments

I took a stab at exposing some musicbrainz metadata if it is present in mpv. This will be useful for anyone who has tagged their music library with Musicbrainz Picard and could be used in some scrobbling programs.

tpeacock19 avatar Aug 09 '22 05:08 tpeacock19

Actually i need to update a bit more with this. I want to cover all the default separator options in Musicbrainz Picard. Which gives the options / ; and /(with space surrounding).

tpeacock19 avatar Aug 09 '22 16:08 tpeacock19

Actually, now that I review this I believe that the metadata should not be separated. That would be a job for any program consuming this data. There should be a single value for the Musicbrainz Album/Artist/etc. Id, also including any potential separators. This would make this PR much simpler. If you agree I'll force push some reversions.

tpeacock19 avatar Aug 09 '22 16:08 tpeacock19

Yeah I think that makes sense. To be honest I'm not very familiar with music brainz so if you think that makes sense let's do it!

hoyon avatar Aug 09 '22 19:08 hoyon

Okay I've got a bit more to work on this. Will let you know when it's ready.

tpeacock19 avatar Aug 10 '22 08:08 tpeacock19

Ah apologies for not letting you know but this has been working for me for about a month. Thanks for merging.

tpeacock19 avatar Sep 10 '22 18:09 tpeacock19