Skip to content

package.json: Set license to GPLv3 #43

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

Merged
merged 1 commit into from
Jun 7, 2018
Merged

Conversation

bekicot
Copy link
Contributor

@bekicot bekicot commented Jun 5, 2018

Fixes #41

@bekicot
Copy link
Contributor Author

bekicot commented Jun 5, 2018

@blazeu && @jayvdb, please review

@jayvdb
Copy link
Member

jayvdb commented Jun 6, 2018

Commit message is "GPLV3" which isn't what the commit does.

Copy link
Member

@jayvdb jayvdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message.

@bekicot bekicot changed the title package.json: Set license to GPLV3 package.json: Set license to GPLv3 Jun 7, 2018
package.json Outdated
@@ -3,7 +3,7 @@
"version": "0.0.0",
"private": true,
"description": "Small description for gsoc-prep-tasks goes here",
"license": "MIT",
"license": "GPLv3",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://docs.npmjs.com/files/package.json#license says to use an identifier on https://spdx.org/licenses/

GPLv3 isnt on that list.

@bekicot bekicot force-pushed the update-license branch 2 times, most recently from 3f905f2 to cacaeb8 Compare June 7, 2018 05:50
blazeu
blazeu previously requested changes Jun 7, 2018
package.json Outdated
@@ -3,7 +3,7 @@
"version": "0.0.0",
"private": true,
"description": "Small description for gsoc-prep-tasks goes here",
"license": "MIT",
"license": "GPL-3.0-only",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LICENSE.md said either version 3 of the License, or (at your option) any later version.

Should be or-later, see header of https://spdx.org/licenses/GPL-3.0-or-later.html#licenseText

Although, GPL-3.0-only also said either version https://spdx.org/licenses/GPL-3.0-only.html#licenseText ,
the header is different.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good eyes ... spdx/license-list-XML#610 and spdx/license-list-XML#617 are about this problem.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can ignore that bit, but it will be relevant to any attempt at automating the LICENSE files via moban :/

The important part is that the GPL-3.0-only requires the preamble which says ', version 3.' and this repo LICENSE doesnt have that.

See placement of <standardLicenseHeader> in
https://github.com/spdx/license-list-XML/blob/master/src/GPL-3.0-only.xml
whereas <standardLicenseHeader> is in <optional> in
https://github.com/spdx/license-list-XML/blob/master/src/GPL-3.0-or-later.xml

@jayvdb
Copy link
Member

jayvdb commented Jun 7, 2018

ack 215892c

@jayvdb jayvdb merged commit 215892c into coala:master Jun 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants