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

Fix bulleted list formatting and clarify license #36

Merged
merged 2 commits into from
Mar 30, 2019

Conversation

psvenk
Copy link
Member

@psvenk psvenk commented Mar 28, 2019

GNU has multiple licenses, so "GNU License" is ambiguous. I changed it to "the GNU General Public License, version 3" because that is the license in the file LICENSE.md, but I would recommend changing it to "the GNU General Public License, either version 3, or (at your discretion) any later version" (or even better, "the GNU General Public License, either version 2, or (at your discretion) any later version or later" if you want to be compatible with GPLv2), so that if a GPLv4 license is released, Aspine will be compatible with projects that use GPLv4. Note that "compatible" means that it is legally possible for two projects under different licenses to share code with each other (two-way compatibility in this specific context).

I have not made this change because I have not written a single line of code in Aspine and so do not hold the copyright to the software. If you want to change the wording of the license, you should be able to push directly to my fork because I checked the "Allow edits from maintainers" box.

GNU has multiple licenses, so "GNU License" is ambiguous. I changed it to "GNU GPLv3" because that is the license in the file LICENSE.md, but I would recommend changing it to "GNU GPLv3 or later" (or even better, "GNU GPLv2 or later" if you want to be compatible with GPLv2), so that if a GPLv4 license is released, Aspine will be compatible with projects that use GPLv4. I have not made this change because I do not hold the copyright to the software and do not have the legal right to make such a decision about the license of the software.
@psvenk psvenk closed this Mar 29, 2019
@psvenk
Copy link
Member Author

psvenk commented Mar 29, 2019

I've closed this because there's another modification that I'd like to make before merging into master. I'll re-open it when it's ready.

@psvenk psvenk reopened this Mar 29, 2019
@maxtkc maxtkc merged commit f355b10 into Aspine:master Mar 30, 2019
@psvenk psvenk deleted the patch-1 branch March 30, 2019 01:38
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.

2 participants