-
Notifications
You must be signed in to change notification settings - Fork 6
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
Normalize and merge copyright lines #23
Comments
For Tidal it is more complicated, the
So I'm not sure whether we can or should add © or ℗ (or maybe "© + ℗") to the string if the symbol is missing. What we could do is converting the ASCII variants into symbols. EDIT: Tidal's API docs give this as an example value for So even this makes it clear that it is not clearly for either © or ℗, but rather free to use for the labels / artists. |
I wonder that maybe "copyright" should be a list of strings per provider instead of a single string. Currently it is I think only Spotify which offers explicit distinct values for (P) and (C). But having those as actual separate values will make de-duplication easier. Right now the provider just separates both with a new line. |
Continuing the discussion from #22 (comment)
I fully agree, this is enough for its own PR and it needs more research.
Tidal also has a
copyright
property at the track level by the way, this should also be considered if it is different from the release level coypright. So far they were identical for the releases which I have checked, maybe a compilation has different values there.For starters I have a commit in the
dev
branch which displays the alternative copyright values.When we have more examples we can decide how the release merge algorithm should handle these, one possibility would be to keep all and deduplicate them.
The text was updated successfully, but these errors were encountered: