pyAudioAnalysis icon indicating copy to clipboard operation
pyAudioAnalysis copied to clipboard

Use of eyed3 in pyAudioAnalysis

Open mdinescu opened this issue 3 years ago • 4 comments

This is a question about the license to use pyAudioAnalysis which is currently published under Apache 2.0. The library makes use of the eyeD3 library for reading ID3 tags, which is published under GPLv3.

I was wondering if you'd be willing to accept a pull request to replace the current use of eyeD3 with mutagen which is published under GPLv2, or with STagger which is published under BSD license. Mutagen supports more types of metadata.

Looks like the only use of eyeD3 is in the audioBasicIO class https://github.com/tyiannak/pyAudioAnalysis/blob/a16ca2da5b0a6897c7573a56e747e57cd8e14a3c/pyAudioAnalysis/audioBasicIO.py

I've checked all the other dependencies and the only one that uses GPLv3 is eyeD3.

mdinescu avatar Mar 03 '21 01:03 mdinescu

Indeed, I think it makes sense. Upvoted.

slowglow avatar May 26 '21 08:05 slowglow

That makes perfect sense thank you for that, I've used mutagen in the past, never STagger. So a PR would be great thanx!

tyiannak avatar May 26 '21 08:05 tyiannak

BSD license, if possible, please. Unrestrictive OS licenses go together well. Restrictive OS licenses go together well too. Mixing, not so well. So, please, consider GPL in this case only if implementation is overwhelmingly superior and as an option at that.

slowglow avatar May 26 '21 09:05 slowglow

Sure, let's prefer BSD, I just said I had not used stagger before.

tyiannak avatar May 26 '21 11:05 tyiannak