Skip to content
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

Add reuse compliance #8

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Add reuse compliance #8

merged 3 commits into from
Sep 23, 2024

Conversation

AndyScherzinger
Copy link

@AndyScherzinger AndyScherzinger commented Jul 17, 2024

Adding SPDX header.

@ChristophWurst while adding the headers I realized that:

So I would suggest to change the license of the php wrapper to AGPL3+ or LGPL3+

What do you think @arnebr @ChristophWurst @come-nc @kesselb @MCMic

Signed-off-by: Andy Scherzinger <[email protected]>
Signed-off-by: Andy Scherzinger <[email protected]>
Signed-off-by: Andy Scherzinger <[email protected]>
@AndyScherzinger AndyScherzinger added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested labels Jul 17, 2024
@AndyScherzinger
Copy link
Author

So this PR in itself is "fine" yet the license issue should be clarified/decided

@AndyScherzinger
Copy link
Author

Alternatively the Lib moves to AGPL3+ which is also doable of course

@ChristophWurst
Copy link

Alternatively the Lib moves to AGPL3+ which is also doable of course

I think I'd favor that because we use AGPLv3+ for Nextcloud and its apps too.

https://opensource.stackexchange.com/questions/5637/use-lgpl-code-in-an-agpl-project

@ChristophWurst
Copy link

👍 for converting this project to AGPLv3+.

@arnebr @CarlSchwan @come-nc @kesselb @MCMic do you agree with this?

@CarlSchwan
Copy link

+1

@kesselb kesselb merged commit 6ef186d into main Sep 23, 2024
12 checks passed
@kesselb kesselb deleted the chore/noid/reuseCompliance branch September 23, 2024 08:39
@come-nc
Copy link

come-nc commented Sep 23, 2024

👍 for converting this project to AGPLv3+.

@arnebr @CarlSchwan @come-nc @kesselb @MCMic do you agree with this?

Everyone agreed so license should be changed to AGPLv3+ in a follow-up

@AndyScherzinger
Copy link
Author

See #10 for license change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants