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

Licence is invalid #523

Closed
0xTim opened this issue Dec 7, 2021 · 7 comments
Closed

Licence is invalid #523

0xTim opened this issue Dec 7, 2021 · 7 comments

Comments

@0xTim
Copy link
Member

0xTim commented Dec 7, 2021

The Apache licence added to the project does not contain a copyright year or name of the copyright holder - https://github.com/swift-server/async-http-client/blob/main/LICENSE.txt#L190

This should probably be fixed...

@fabianfett
Copy link
Member

cc @tomerd

@Lukasa
Copy link
Collaborator

Lukasa commented Dec 7, 2021

We don't need @tomerd's help here, we can just update that file. Incidentally I don't believe that it renders the license invalid in any way, it's just a bit less convenient to copy into forks.

@0xTim
Copy link
Member Author

0xTim commented Dec 7, 2021

Who's the code owner here? Swift, Apple or the SSWG? I'm only asking as we need to add a licence for the VSCode extension which is how I stumbled upon this

@Lukasa
Copy link
Collaborator

Lukasa commented Dec 7, 2021

Copyright (c) 2019-2021 Apple Inc. and the AsyncHTTPClient project authors

@weissi
Copy link
Contributor

weissi commented Dec 7, 2021

@0xTim I think the LICENSE.txt file is actually supposed to have the placeholders.

The quoted line is after the END OF TERMS AND CONDITIONS in a section called APPENDIX: How to apply the Apache License to your work. which I believe is just there to tell you what you need to do in every file.

And from what I can tell, each file in AHC has the correct information (ie. the placeholders replaced with the real info).

@0xTim
Copy link
Member Author

0xTim commented Dec 7, 2021

Hmm maybe you're right, in which case ignore me then!

@ktoso
Copy link
Contributor

ktoso commented Dec 8, 2021

I think same as @weissi -- this is just the section of the license that says "this is how you apply it" -- the actual copyright headers in the actual files have the right info.

We can confirm with some actual legal authority; but another point of reference is Akka and Netty, both of which do the same thing: https://github.com/akka/akka/blob/main/LICENSE https://github.com/netty/netty/blob/4.1/LICENSE.txt - the file is just the template with placeholders.

I believe this/these issue(s) should be closed with no action taken.

@0xTim 0xTim closed this as completed Dec 8, 2021
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

5 participants