Write lyrics to id3 tag

[dmo60]

Add an option to write the lyrics to the lyrics tag of the audio file. Python mutagen can be used for it.

[dmo60]

I tried using mutagen and it was working quite well (you can find it on the experimental branch).

Unfortunatly, Rhythmbox 3 uses Python 3 and mutagen is not available there, so I won't put any more efforts in this. Perhaps I will find another easy way to do it someday.

[Hunsu]

I have found two library that support python 3. https://github.com/LordSputnik/mutagen and https://code.google.com/p/stagger/.

[dmo60]

Cool, the first one looks promising. I might try that. It's not my highest priority at the moment, but I'm definitely planning to implement this someday!

[Hunsu]

I planing to do it too! Have you written some code that could help me as you are saying that you used mutagen before?

[fossfreedom]

me again - mutagenx works very well - the syntax is identical to python2 mutagen and I would warmly recommend using this. I use this on coverart here - the same code works on both python 2 based RB2.96-2.99 and python3 based RB3

• https://github.com/fossfreedom/coverart-search-providers/blob/master/coverart_search_tracks.py

[dmo60]

You can find my attempt using mutagen for python 2 in the experimental branch, mainly here:
https://github.com/dmo60/lLyrics/blob/experimental/lLyrics/Util.py

[Hunsu]

I have implemented this and the changed the search method. The search method as next :

• Look if there's lyrics in audio tag (only for mp3 and ogg files)
• if not search for lyrics sources with Google (in future we may search others searcher)
• We use an xml file that contains lyrics sources to parse lyrics from the source.

The xml file contains the informations :

• The source name
• The name of the tag that contains lyrics (eg. div.lyrics)
• The tag number if there's not a specific tag for lyrics
• The start string as you used it in your lyrics sources
• The end string.
  The lLyrics.py code must be changed, I will do it when I have some time. There's a problem the xml file must be in home folder. When I put it in lLyrics folder it doesn't work. I will add others sources later. What about the downloadAllLyrics I implemented? have you take a look at it?

the code is here https://drive.google.com/file/d/0B25ViosYm6ByY3l5VHJnVkdjQ28/edit?usp=sharing

Edit :
I have made many changes to this code. I think I could give the final version next week.

[dmo60]

First, thanks for your work! To be honest, these are some big changes and I currently don't have the time to review them all. I have had a look at your downloadAllLyrics and had some small issues, but I don't remember them in detail.

It would be easier for me to merge your changes, if you could submit a pull request: https://help.github.com/articles/using-pull-requests

I'm looking forward to your final version and I hope I will find some time to test it properly. I will give you feedback as soon as I can.

[Hunsu]

So to make a pull request I must fork you project? I will do it when I have some time. Are you ok with the new search method?

[CreamyCookie]

That feature would be super amazing! 👍

It would also be nice to show the lyrics that are included in the tags (if the lyrics couldn't be found).