florian-grond / SC-HOA

HOA wrapper classes for SuperCollider based on https://github.com/sekisushai/ambitools
GNU General Public License v3.0
32 stars 7 forks source link

SC-HOA isn't licensed #39

Closed patrickdupuis closed 3 years ago

patrickdupuis commented 4 years ago

SC-HOA isn't licensed and it needs to be. The ambitools project that it is based on is GPLv2 only (as opposed to GPLv2 or later), and the IEM resources that it includes are GPLv3. There's a problem with these two licenses.

In order to add a license to this repo, the above needs to be sorted out, and all contributors to SC-HOA must agree on the license.

patrickdupuis commented 4 years ago

Actually, it's really unclear what the ambitools license is. The FAUST generated C++ files in the SuperCollider folder confusingly say GPLv2 or later and GPLv3 or later. The audio files in the FIR folder are probably licensed like the repo (GPLv2 only?).

What parts of ambitools does SC-HOA use exactly?

florian-grond commented 4 years ago

I just asked Pierre Lecomte, the author of ambitools and will get back to you as soon as I know more.

nicobou commented 4 years ago

Hi @florian-grond any update on this ?

florian-grond commented 4 years ago

I have contacted Piere Lecomte, but unfortunately, he has not responded yet.

Florian

https://www.linkedin.com/in/floriangrond/

On Tue, Jan 14, 2020 at 1:27 PM Nicolas Bouillot notifications@github.com wrote:

Hi @florian-grond https://github.com/florian-grond any update on this ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/florian-grond/SC-HOA/issues/39?email_source=notifications&email_token=AAKUTRROSIMO2RVE6OJEG7TQ5X7YBA5CNFSM4J2BWPEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI5UFJI#issuecomment-574309029, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKUTRQMDUNTCCNSIUSSLP3Q5X7YBANCNFSM4J2BWPEA .

nicobou commented 4 years ago

@florian-grond if you set your repo as GPLv3 or later, then it will fit with IEM resources and probably also with ambotools since GPL2 seems to be compatible with later version (?).

If you agree with this, you only need to add this file to the root of your repo: https://www.gnu.org/licenses/gpl-3.0.md

but the file should be called LICENSE.md.

florian-grond commented 4 years ago

done https://www.linkedin.com/in/floriangrond/

On Wed, Jan 15, 2020 at 8:52 AM Nicolas Bouillot notifications@github.com wrote:

@florian-grond https://github.com/florian-grond if you set your repo as GPLv3 or later, then it will fit with IEM resources and probably also with ambotools since GPL2 seems to be compatible with later version (?).

If you agree with this, you only need to add this file to the root of your repo: https://www.gnu.org/licenses/gpl-3.0.md

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/florian-grond/SC-HOA/issues/39?email_source=notifications&email_token=AAKUTRVNIT7BC4B3ELK7UIDQ545MZA5CNFSM4J2BWPEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJBAFXI#issuecomment-574751453, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKUTRR27LC6JNRUDUFTQ5DQ545MZANCNFSM4J2BWPEA .

patrickdupuis commented 4 years ago

if you name the file LICENSE.md github will see it better. You should also make a new release tag 1.4.0.

florian-grond commented 4 years ago

done https://www.linkedin.com/in/floriangrond/

On Fri, Jan 17, 2020 at 9:13 AM Patrick Dupuis notifications@github.com wrote:

if you name the file LICENSE.md github will see it better. You should also make a new release tag 1.4.0.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/florian-grond/SC-HOA/issues/39?email_source=notifications&email_token=AAKUTRW7A2F6VQXKFXMS55LQ6HRLTA5CNFSM4J2BWPEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJILQHQ#issuecomment-575715358, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKUTRXLNMY5LSY2SSTCSRLQ6HRLTANCNFSM4J2BWPEA .

thibaudk commented 3 years ago

closing with 066c7f9