Skip to content

preliminary work for SoundTouch BPM detection#6

Closed
enetheru wants to merge 6 commits intoUltraStar-Deluxe:BASS_removedfrom
enetheru:bpmSoundTouch
Closed

preliminary work for SoundTouch BPM detection#6
enetheru wants to merge 6 commits intoUltraStar-Deluxe:BASS_removedfrom
enetheru:bpmSoundTouch

Conversation

@enetheru
Copy link
Copy Markdown

removed dependency on aubio

This pull request is just so you know i'm looking into it. I will update it as i get further along.

@basisbit
Copy link
Copy Markdown
Member

you might want to check the licensing stuff. I am not sure if one is allowed to use a LGPL library in a GPLv2+ application. If that actually is allowed, a link to some good explaining article would be nice 😃

a very light shim over the top of QAudioDecoder interfacing it with SoundTouch::BPMDetect class
@enetheru
Copy link
Copy Markdown
Author

@basisbit lgpl can be used in proprietary software so long as you arent statically linking to it. if it remains a separate library you're sweet and is the entire purpose of LGPL vs GPL

https://stackoverflow.com/questions/10130143/gpl-lgpl-and-static-linking is no1 on google.

@enetheru
Copy link
Copy Markdown
Author

also the QT libraries are lgplv3 so thats already happening.. :P

@basisbit
Copy link
Copy Markdown
Member

the linked stackoverflow thread does not answer my question at all. UltraStar Creator is licensed under GPLv2+, SoundTech is licensed under LGPL. Please try again.

@enetheru
Copy link
Copy Markdown
Author

@basisbit I'm sorry that you dont think that answer's your question. Since you are the one who is unsure please go and do the research yourself. I however, am sure that it's perfectly legal to link to an LGPL library from anywhere so long as it's only linked dynamically it will not pollute the rest of the source. the terms of ultrastars licence has zero bearing on SoundTouch's code since they do not have any dependency or link to us, that's not how licensing requirements work.

@AlexanderS
Copy link
Copy Markdown
Member

https://softwareengineering.stackexchange.com/a/342158

@enetheru enetheru closed this by deleting the head repository Feb 16, 2026
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.

3 participants