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

Missing SPDX identifier data #62

Open
17 of 23 tasks
goneall opened this issue Sep 5, 2021 · 14 comments · Fixed by #65 · May be fixed by #84
Open
17 of 23 tasks

Missing SPDX identifier data #62

goneall opened this issue Sep 5, 2021 · 14 comments · Fixed by #65 · May be fixed by #84
Labels

Comments

@goneall
Copy link
Contributor

goneall commented Sep 5, 2021

The following SPDX identifiers are not in this repo/JSON file but are listed on the OSI website:

  • 0BSD
  • BSD-2-Clause-Patent
  • BSD-1-Clause
  • BSD-3-Clause-LBNL
  • CAL-1.0
  • EPL-2.0
  • EUPL-1.2
  • CECILL-2.1
  • OLDAP-2.8
  • PHP-3.01
  • MIT-0

The SPDX license ID on the website is incorrect for the following:

  • Licence Libre du Québec – Réciprocité forte (LiLiQ-R+) version 1.1 (LiliQ-R+) - should be (LiLiQ-Rplus-1.1)
  • Licence Libre du Québec – Réciprocité (LiLiQ-R) version 1.1 (LiliQ-R) - should be (LiLiQ-R-1.1)
  • Licence Libre du Québec – Permissive (LiLiQ-P) version 1.1 (LiliQ-P) - should be (LiLiQ-P-1.1)
  • Universal Permissive License (UPL) - should be (UPL-1.0)

The following licenses are listed by name on the OSI website but are missing the SPDX identifiers:

  • CERN Open Hardware Licence Version 2 - Permissive (CERN-OHL-P-2.0)
  • CERN Open Hardware Licence Version 2 - Weakly Reciprocal (CERN-OHL-S-2.0)
  • CERN Open Hardware Licence Version 2 - Strongly Reciprocal (CERN-OHL-W-2.0)
  • Mulan Permissive Software License v2 (MulanPSL-2.0)
  • OSET Public License version 2.1 (OSET-PL-2.1)
  • Upstream Compatibility License v1.0 (UCL-1.0)
  • The Unlicense (Unlicense)
  • Unicode Data Files and Software License (Unicode-DFS-2016)

Although not an inconsistency between the website and this file, I would also like to add some additional SPDX identifiers to the GPL family of licenses. Per request from the Free Software Foundation a couple years ago, we added license identifiers with a suffix "-or-later" to the GPL and LGPL family of licenses. If would be nice if we added those identifiers to this JSON representation. It may also make sense to add them to the website.

@webmink
Copy link
Member

webmink commented Sep 26, 2021

Anticipating some more pull requests for the remaining items raised, which I'll add as they arise.

@goneall
Copy link
Contributor Author

goneall commented Sep 30, 2021

@webmink I added PR #69

I'm wondering if multiple PR's are going to create merge conflicts. I'm going to wait until 0BSD is merged before adding any more PR's in case the current process causes conflicts.

@webmink
Copy link
Member

webmink commented Oct 1, 2021

OK. Hopefully we'll be caught up in the next few days!

@goneall
Copy link
Contributor Author

goneall commented Oct 14, 2021

@paultag You may want to re-open this issue since there are several additional ID's that need to be added. I'll work on those next now that we have PR #65 merged.

@paultag paultag reopened this Oct 14, 2021
goneall added a commit to goneall/licenses that referenced this issue Nov 7, 2021
License text copied from the OSI website.

License name change from:
"Eclipse Public License version 2.0"
to:
"Eclipse Public License Version 2.0"

based on errors reported from compile.py.

This license is listed in issue OpenSourceOrg#62

Signed-off-by: Gary O'Neall <[email protected]>
@goneall goneall mentioned this issue Nov 7, 2021
goneall added a commit to goneall/licenses that referenced this issue Nov 7, 2021
License text matched the license text from SPDX.

License name was change from:
"European Union Public License, version 1.2"
to:
"European Union Public License, Version 1.2"
based on errors reported in compile.py

This license is listed in issue OpenSourceOrg#62

Signed-off-by: Gary O'Neall <[email protected]>
goneall added a commit to goneall/licenses that referenced this issue Nov 8, 2021
License text and title copied from OSI Page
This license is listed in issue OpenSourceOrg#62

Signed-off-by: Gary O'Neall <[email protected]>
goneall added a commit to goneall/licenses that referenced this issue Nov 8, 2021
License text and name copied from OSI web page.

This license is listed in issue OpenSourceOrg#62

Signed-off-by: Gary O'Neall <[email protected]>
goneall added a commit to goneall/licenses that referenced this issue Nov 8, 2021
Copied text from OSI website.  License name matches SPDX name.

This license is listed in issue OpenSourceOrg#62

Signed-off-by: Gary O'Neall <[email protected]>
@goneall
Copy link
Contributor Author

goneall commented Nov 8, 2021

I just put in PR's for all remaining SPDX identifiers are not in this repo/JSON file but are listed on the OSI website.

Note CECILL-2.1 was already there - so no need for a PR.

Once these are merged, I'll create PRs for the remaining issues.

@webmink
Copy link
Member

webmink commented Nov 8, 2021

Since licensing matters are ultimately overseen for OSI by @pchestek I'm adding her to all of them as an approver. Once she's given us the nod either I or @paultag will merge them.

@goneall
Copy link
Contributor Author

goneall commented May 30, 2022

@paultag @webmink Several of the licenses present on the website with missing SPDX id's are not present in this repo.

Would you like me to create a PR which adds the licenses along with the SPDX ID's missing from the website or did you want to sync up this repo with the website content?

Below is the list of licenses I found on the website without corresponding text in this repo:

CERN Open Hardware Licence Version 2 - Permissive (CERN-OHL-P-2.0)
CERN Open Hardware Licence Version 2 - Weakly Reciprocal (CERN-OHL-S-2.0)
CERN Open Hardware Licence Version 2 - Strongly Reciprocal (CERN-OHL-W-2.0)
Mulan Permissive Software License v2 (MulanPSL-2.0)
Upstream Compatibility License v1.0 (UCL-1.0)
The Unlicense (Unlicense)
Unicode Data Files and Software License (Unicode-DFS-2016)

@richardfontana
Copy link

The SPDX license ID on the website is incorrect for the following:

@goneall in general it is incorrect to say these are errors. I am responsible for a few of those cases as I was on the OSI board at the time of the license approval. The case I remember pretty clearly is that of the Licence Libre du Québec licenses. The abbreviations you mention are the ones used by the license steward in the course of making their license submission and (IIRC) reflected in the license texts themselves. At the time, we weren't trying to root out all possible forms of referring to OSI-approved licenses in abbreviated form other than using SPDX identifiers. It's sort of like saying any mention of "GPLv2" is incorrect because "GPLv2" is not an SPDX identifier. Different or alternative system of license abbreviation != "incorrect". It's possible that since the time I left the OSI board the OSI has chosen to more ambitiously or consistently promote use of SPDX identifiers.

@goneall
Copy link
Contributor Author

goneall commented Dec 12, 2022

@goneall in general it is incorrect to say these are errors.

@richardfontana I'm not commenting on the OSI ID's, I'm specifically referring to the SPDX identifiers on the OSI website.

On the License Listed Alphabetically, the following description of the list is provided:

The following licenses have been approved by the OSI. The parenthesized expression following a license name is its SPDX short identifier (if one exists).

Looking at the SPDX Id's specifically, I found the inconsistency with the SPDX ID's on the SPDX listed license pages.

For example, the following appears on the OSI page:
Licence Libre du Québec – Réciprocité forte (LiLiQ-R+) version 1.1 (LiliQ-R+)

while the SPDX license list has:
Licence Libre du Québec – Réciprocité forte version 1.1 | LiLiQ-Rplus-1.1

Since the OSI web page specifically mentions the SPDX ID for the ID in parenthesis following the name, I would expect the ID to match the SPDX listed license ID. Let me know if you disagree.

@richardfontana
Copy link

Since the OSI web page specifically mentions the SPDX ID for the ID in parenthesis following the name, I would expect the ID to match the SPDX listed license ID. Let me know if you disagree.

@goneall you are correct, I misunderstood. Given that you can see the care with which BSD+Patent is handled on this list (with "BSD-2-Clause-Patent" as the parenthesized identifier) I can't explain what happened with the other cases.

@kopeboy
Copy link

kopeboy commented Apr 15, 2023

Any update on this?! I was trying to import the list of OSI-approved licenses to my platform but the link for the JSON in the readme (which is not even https) is not consistent with the info on OSI website.. this is either outdated or a critical bug imho.
I liked the "keywords" you provide to be able to filter, but I guess importing from SPDX is much safer since their repository seems maintained and api + website implementations are consistent.

@goneall
Copy link
Contributor Author

goneall commented Apr 15, 2023

@kopeboy - I was waiting for PR #82 to be merged before making any additional changes.

@webmink - Any progress on the PR?

@webmink
Copy link
Member

webmink commented Apr 16, 2023

Done.

@goneall
Copy link
Contributor Author

goneall commented Apr 17, 2023

Thanks @webmink

@kopeboy - I have some major release for some SPDX tools this week, but I'll try to get to the other PR's.

BTW - feel free to create PR's for these yourself if you'd like to move these along more quickly - we're all just volunteers ;)

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