Skip to content

Fix Issue 104: supply original BSD-style license#105

Closed
bbadour wants to merge 1 commit intomoby:masterfrom
bbadour:gobsd
Closed

Fix Issue 104: supply original BSD-style license#105
bbadour wants to merge 1 commit intomoby:masterfrom
bbadour:gobsd

Conversation

@bbadour
Copy link
Contributor

@bbadour bbadour commented Mar 14, 2026

I located the original LICENSE file content that accompanied dictionary.go and appended it to the LICENSE file here.

I located the original LICENSE file content that accompanied
dictionary.go and appended it to the LICENSE file here.

Signed-off-by: Bob Badour <bbadour@google.com>
@thaJeztah
Copy link
Member

Adding it inside the Apache license is probably confusing; the project is Apache licensed (which allows including MIT licensed code); given that effectively the spdy directory as a whole was MIT licensed, what we can do (and have done elsewhere) is to add a copy of the MIT LICENSE file in that subdirectory.

@dims
Copy link
Collaborator

dims commented Mar 14, 2026

i agree with @thaJeztah

@bbadour
Copy link
Contributor Author

bbadour commented Mar 15, 2026

The lawyers where I work appreciate having all license texts that apply to a project in a single file at the root of the project. The header just says "the LICENSE". Sometimes they specify it's in the root; this one doesn't.

I have no objection if someone wants to break it out into a separate file. I can work with it where-ever you put it.

@bbadour
Copy link
Contributor Author

bbadour commented Mar 15, 2026

P.S. If the other contents of the spdy/ directory are MIT licensed, that's a third license that might need mentioning somewhere. I didn't notice any mention of MIT in any of those files.

@thaJeztah
Copy link
Member

P.S. If the other contents of the spdy/ directory are MIT licensed, that's a third license that might need mentioning somewhere. I didn't notice any mention of MIT in any of those files.

Ah, sorry, I posted from my phone and meant BSD, not MIT 🙈

I also did a bit of searching, and opened #106 as alternative; happy to hear if that works for you; I added you as co-author, but let me know if you prefer to not have that 👍

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.

3 participants