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

OpenSSL copyright? #4

Closed
levitte opened this issue Feb 1, 2021 · 6 comments
Closed

OpenSSL copyright? #4

levitte opened this issue Feb 1, 2021 · 6 comments

Comments

@levitte
Copy link
Contributor

levitte commented Feb 1, 2021

There are a number of files in oqsprov/ that have the OpenSSL copyright boilerplate, which gives copyright to the OpenSSL Authors, which I understand is historical, since this was previously a patchset for OpenSSL proper.

I don't think that's something you want at this point, and I can't see anything wrong with simply removing that boilerplate. What you still have in there that could remind anyone of the built-in OpenSSL providers, it's mostly bread-and-butter lines that are only vaguely similar.

@baentsch
Copy link
Member

baentsch commented Feb 2, 2021

ACK. We'd probably take the (lib)OQS marking route. Looking at that again, though, // SPDX-License-Identifier: Apache-2.0 AND MIT is a "bit terse".... @dstebila Suggestions?

@dstebila
Copy link
Member

dstebila commented Feb 2, 2021

In liboqs for files we created, we did not put individual license information in each file beyond the SPDX-License-Identifier header, but there was a LICENSE.txt at the root which contained that information. I think that would be okay here.

baentsch added a commit that referenced this issue Feb 3, 2021
@baentsch
Copy link
Member

baentsch commented Feb 3, 2021

Resolved with d6223cb

@baentsch baentsch closed this as completed Feb 3, 2021
@levitte
Copy link
Contributor Author

levitte commented Feb 3, 2021

You missed a spot 😉

/*
* Copyright 2016-2020 The OpenSSL Project Authors. All Rights Reserved.
*
* Licensed under the Apache License 2.0 (the "License"). You may not use
* this file except in compliance with the License. You can obtain a copy
* in the file LICENSE in the source distribution or at
* https://www.openssl.org/source/license.html
*/

baentsch added a commit that referenced this issue Feb 4, 2021
@baentsch
Copy link
Member

baentsch commented Feb 4, 2021

You missed a spot

Thanks for the catch. And it was more than one. Lesson: No change too small to not botch it :-( Hopefully #5 gets this right. I won't bother you to review it (but won't stop you :-)

dstebila pushed a commit that referenced this issue Feb 4, 2021
@levitte
Copy link
Contributor Author

levitte commented Feb 4, 2021

[ahem] I might look over your virtual shoulder a little now and then. Just the fact that others are already playing with external provider is exciting

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

No branches or pull requests

3 participants