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

Add BSDplusPatent license (SPDX ID BSD-2-Clause-Patent) #69

Merged
merged 2 commits into from
Oct 14, 2021

Conversation

goneall
Copy link
Contributor

@goneall goneall commented Sep 30, 2021

Partially resolves issue #62

Note that the license name was updated to match the OSI website.

License text was an exact match between the OSI and SPDX website.

@goneall goneall mentioned this pull request Sep 30, 2021
23 tasks
@webmink webmink requested a review from pchestek October 1, 2021 10:30
@webmink
Copy link
Member

webmink commented Oct 1, 2021

@paultag @zacchiro Can you educate me on that required CI check please? Otherwise I'd merge it when @pchestek has approved.

@zacchiro
Copy link
Contributor

zacchiro commented Oct 1, 2021

I'm not myself familiar with the CI configuration of this repo, so I can't help on that front.
But FWIW I've nothing against merging this PR.

@paultag
Copy link
Collaborator

paultag commented Oct 13, 2021

CI looks broken, could be the version of Python was never updated, could be our Travis account is offline (not sure).

When I run what the CI runs locally after reviewing the change, this all looks good

90 records written out
 082 licenses contain scheme SPDX (91.111111%)
 043 licenses contain scheme Trove (47.777778%)
 022 licenses contain scheme DEP5 (24.444444%)
 090 licenses contain tag osi-approved (100.000000%)
 050 licenses contain tag discouraged (55.555556%)
 011 licenses contain tag redundant (12.222222%)
 003 licenses contain tag miscellaneous (3.333333%)
 026 licenses contain tag non-reusable (28.888889%)
 009 licenses contain tag obsolete (10.000000%)
 011 licenses contain tag popular (12.222222%)
 005 licenses contain tag permissive (5.555556%)
 004 licenses contain tag retired (4.444444%)
 005 licenses contain tag special-purpose (5.555556%)
 007 licenses contain tag copyleft (7.777778%)
 003 licenses contain tag international (3.333333%)

@paultag paultag self-assigned this Oct 13, 2021
@goneall
Copy link
Contributor Author

goneall commented Oct 13, 2021

CI looks broken, could be the version of Python was never updated, could be our Travis account is offline (not sure).

@paultag You may want to look into moving over to Github actions - I switched from Travis to Github when Travis changed some of their configuration requirements and it has worked well for me. If you want an example of an action that runs a python script on pushes, you can check out the spdx-spec github action.

@paultag
Copy link
Collaborator

paultag commented Oct 13, 2021

@goneall yeah that was going to be my plan; I'm stuck without permissions ATM, I'll get around to it as soon as I can

@webmink
Copy link
Member

webmink commented Oct 13, 2021

@paultag I gave you Maintain access, tell me what you need and it's yours (well, within physics and reason)

@paultag
Copy link
Collaborator

paultag commented Oct 13, 2021

@webmink I still can't see the right knobs, I think I need my admin bits back :)

@webmink
Copy link
Member

webmink commented Oct 13, 2021

@paultag You've got it now!

@paultag paultag merged commit eead2d4 into OpenSourceOrg:master Oct 14, 2021
@goneall goneall deleted the bsd2clausepatent branch November 7, 2021 18:57
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.

5 participants