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

dist: fix incorrect license on rpm & deb packages #564

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

syuu1228
Copy link
Contributor

No description provided.

@syuu1228 syuu1228 requested a review from yaronkaikov December 23, 2024 11:20
@syuu1228 syuu1228 marked this pull request as ready for review December 23, 2024 11:21
@syuu1228 syuu1228 force-pushed the change_license_to_agpl3 branch from 1abbb39 to d9aa7bd Compare December 23, 2024 11:23
@syuu1228
Copy link
Contributor Author

I found that since we switched our license from AGPL-3.0 to Apache at e4116ac, we mistakenly not updated dist/debian/debian/copyright.
So it sill AGPL-3.0 until now, we don't need to update dist/debian to switching license.

@syuu1228
Copy link
Contributor Author

@yaronkaikov
Copy link
Collaborator

@avikivity @mykaul please have a look

@yaronkaikov yaronkaikov requested a review from mykaul December 23, 2024 14:57
@avikivity
Copy link
Member

Why to AGPL? Better to Apache license, we don't want to restrict use here.

@syuu1228
Copy link
Contributor Author

I opened this PR because of the request from @yaronkaikov, which was:

Do you have any idea how to modify the license for scylla-machine-image from AGPL to APACHE ?
Can you update about the license change for scylla-machine-image?

So I think current patch is doing opposite.
@avikivity We want use Apache-2.0 instead of AGPL-3, right?

Current license is already Apache-2.0 so it's basically nothing to do here.
But I found one exception:

dist/debian/debian/copyright is still says this code is licensed under AGPL-3.
So I guess we should change this file to Apache-2.

@avikivity
Copy link
Member

Yes let's change the leftover to Apache2

It wasn't updated correctly, should be "Apache-2.0".
@syuu1228 syuu1228 force-pushed the change_license_to_agpl3 branch from d9aa7bd to ff64db2 Compare February 12, 2025 13:37
@syuu1228 syuu1228 changed the title Switch license to AGPL-3.0 dist: fix incorrect license on rpm & deb packages Feb 12, 2025
@syuu1228
Copy link
Contributor Author

Fixed incorrect license on .deb and .rpm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants