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

Inconsitency in file license-list-data/json/licenses.json #1676

Closed
vargenau opened this issue Oct 13, 2022 · 7 comments
Closed

Inconsitency in file license-list-data/json/licenses.json #1676

vargenau opened this issue Oct 13, 2022 · 7 comments
Labels
XML markup change potential change or addition to XML markup in license
Milestone

Comments

@vargenau
Copy link
Contributor

In file license-list-data/json/licenses.json you have:

    {
      "reference": "https://spdx.org/licenses/GPL-3.0.html",
      "isDeprecatedLicenseId": true,
      "detailsUrl": "https://spdx.org/licenses/GPL-3.0.json",
      "referenceNumber": 242,
      "name": "GNU General Public License v3.0 only",
      "licenseId": "GPL-3.0",
      "seeAlso": [
        "https://www.gnu.org/licenses/gpl-3.0-standalone.html",
        "https://opensource.org/licenses/GPL-3.0"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

    {
      "reference": "https://spdx.org/licenses/GPL-3.0-only.html",
      "isDeprecatedLicenseId": false,
      "detailsUrl": "https://spdx.org/licenses/GPL-3.0-only.json",
      "referenceNumber": 274,
      "name": "GNU General Public License v3.0 only",
      "licenseId": "GPL-3.0-only",
      "seeAlso": [
        "https://www.gnu.org/licenses/gpl-3.0-standalone.html",
        "https://opensource.org/licenses/GPL-3.0"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

In short, you have the same "name" for both the deprecated license and the replacement license.

This is true for:

             "name": "GNU General Public License v1.0 only",
             "name": "GNU General Public License v1.0 or later",
             "name": "GNU General Public License v2.0 only",
             "name": "GNU General Public License v2.0 or later",
             "name": "GNU General Public License v3.0 only",
             "name": "GNU General Public License v3.0 or later",
             "name": "GNU Lesser General Public License v2.1 only",
             "name": "GNU Lesser General Public License v3.0 only",
             "name": "GNU Lesser General Public License v3.0 or later",
             "name": "GNU Library General Public License v2 only",
             "name": "GNU Library General Public License v2 or later",
             "name": "Standard ML of New Jersey License",

But it is not the case for other deprecated licenses.

For example, you have:

   {
      "reference": "https://spdx.org/licenses/LGPL-2.1-or-later.html",
      "isDeprecatedLicenseId": false,
      "detailsUrl": "https://spdx.org/licenses/LGPL-2.1-or-later.json",
      "referenceNumber": 31,
      "name": "GNU Lesser General Public License v2.1 or later",
      "licenseId": "LGPL-2.1-or-later",
      "seeAlso": [
        "https://www.gnu.org/licenses/old-licenses/lgpl-2.1-standalone.html",
        "https://opensource.org/licenses/LGPL-2.1"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

and

    {
      "reference": "https://spdx.org/licenses/LGPL-2.1+.html",
      "isDeprecatedLicenseId": true,
      "detailsUrl": "https://spdx.org/licenses/LGPL-2.1+.json",
      "referenceNumber": 320,
      "name": "GNU Library General Public License v2.1 or later",
      "licenseId": "LGPL-2.1+",
      "seeAlso": [
        "https://www.gnu.org/licenses/old-licenses/lgpl-2.1-standalone.html",
        "https://opensource.org/licenses/LGPL-2.1"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

Also:

   {
      "reference": "https://spdx.org/licenses/AGPL-3.0-only.html",
      "isDeprecatedLicenseId": false,
      "detailsUrl": "https://spdx.org/licenses/AGPL-3.0-only.json",
      "referenceNumber": 343,
      "name": "GNU Affero General Public License v3.0 only",
      "licenseId": "AGPL-3.0-only",
      "seeAlso": [
        "https://www.gnu.org/licenses/agpl.txt",
        "https://opensource.org/licenses/AGPL-3.0"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

and

    {
      "reference": "https://spdx.org/licenses/AGPL-3.0.html",
      "isDeprecatedLicenseId": true,
      "detailsUrl": "https://spdx.org/licenses/AGPL-3.0.json",
      "referenceNumber": 70,
      "name": "GNU Affero General Public License v3.0",
      "licenseId": "AGPL-3.0",
      "seeAlso": [
        "https://www.gnu.org/licenses/agpl.txt",
        "https://opensource.org/licenses/AGPL-3.0"
      ],
      "isOsiApproved": true,
      "isFsfLibre": true
    },

I would expect that you have the same "name" for both the deprecated license and the replacement license.

@vargenau
Copy link
Contributor Author

Hi @goneall

Could you please have a look at this issue?

@goneall
Copy link
Member

goneall commented Oct 25, 2022

@vargenau The name is getting pulled from the license XML name attribute on the license element.

This is more of a policy question as to whether names should be the same or different and if there even needs to be a consistency on the names for deprecated license names.

Once the policy is decided, the license XML's may need to be updated. Since the names are more informative and are not required to be unique, I think it would be OK to update the names for existing licenses.

@vargenau
Copy link
Contributor Author

Hi @goneall

I have created pull requests: #1710 #1711 #1712 #1713 #1714 #1715 #1716

@goneall
Copy link
Member

goneall commented Nov 16, 2022

Thanks @vargenau - the license-list-data should update within an hour of those PR's being merged.

@vargenau
Copy link
Contributor Author

vargenau commented Dec 8, 2022

Hi @goneall

Could you please merge these pull requests?

Thank you

@goneall
Copy link
Member

goneall commented Dec 8, 2022

@jlovejoy @swinslow @zvr - Could I get one more review on the above referenced PR's - or let me know if you'd like me to just merge.

@swinslow swinslow added the XML markup change potential change or addition to XML markup in license label Dec 24, 2022
@swinslow swinslow added this to the 3.20 milestone Dec 24, 2022
@swinslow
Copy link
Member

Thanks @vargenau for the attention to detail on these!

Based on that, I'm going to go ahead and close this issue after closing #1712 and #1713. Thanks all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
XML markup change potential change or addition to XML markup in license
Projects
None yet
Development

No branches or pull requests

3 participants