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

Dual licensing #49

Closed
dstebila opened this issue Apr 17, 2023 · 6 comments · Fixed by #55
Closed

Dual licensing #49

dstebila opened this issue Apr 17, 2023 · 6 comments · Fixed by #55

Comments

@dstebila
Copy link

In liboqs we've gotten some feedback (open-quantum-safe/liboqs#1437) that the CC0 license is a barrier to adoption by some distributions. Would SPHINCS+ be willing to consider dual licensing to add another license? See also open-quantum-safe/liboqs#1388 (comment) for comments from the Fedora team.

@dstebila
Copy link
Author

Just checking to see if there's been any progress on this front.

@bwesterb
Copy link
Contributor

bwesterb commented May 2, 2023

The team seems happy to accommodate Fedora, but we're still discussing which exact combination of licenses to use.

@dstebila
Copy link
Author

dstebila commented Jul 5, 2023

Hi Bas, were you able to reach a conclusion on this?

@MrPugh
Copy link
Contributor

MrPugh commented Jul 6, 2023

I added a branch 'license' to tackle the issue; please check and comment.

@MrPugh MrPugh linked a pull request Jul 6, 2023 that will close this issue
@MrPugh
Copy link
Contributor

MrPugh commented Jul 6, 2023

I am not sure how to handle the NIST files:

ref/PQCgenKAT_sign.c
ref/rng.c
ref/rng.h

@dstebila
Copy link
Author

dstebila commented Jul 6, 2023

It shouldn't be a problem to leave those as is, and make a comment in the README or LICENSE file that those files retain their original license. Downstream projects like OQS won't actually make use of those files in compiled libraries, so the resulting binaries will still be cleanly licensed.

@MrPugh MrPugh closed this as completed in #55 Jul 7, 2023
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 a pull request may close this issue.

3 participants