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

Please consider including other licenses mentioned in LICENSE.txt #565

Closed
musicinmybrain opened this issue Sep 17, 2022 · 3 comments · Fixed by #584
Closed

Please consider including other licenses mentioned in LICENSE.txt #565

musicinmybrain opened this issue Sep 17, 2022 · 3 comments · Fixed by #584
Labels
documentation Documentation

Comments

@musicinmybrain
Copy link
Contributor

musicinmybrain commented Sep 17, 2022

What did you do?

Looked at LICENSE.txt.

What did you expect to happen?

All required license text is available.

What actually happened?

The copyright statement and license text for the main BSD-3-Clause license are present, but the file mentions that code is included or derived from libraries with different licenses.

Portions of code from MODP_ASCII - Ascii transformations (upper/lower, etc)
https://github.com/client9/stringencoders
Copyright (c) 2007  Nick Galbreath -- nickg [at] modp [dot] com. All rights reserved.

Checking https://github.com/client9/stringencoders, the MIT license appears to apply. (See my follow-up comment; this code appears to be used under a BSD-3-Clause license.)

Numeric decoder derived from from TCL library
https://opensource.apple.com/source/tcl/tcl-14/tcl/license.terms
 * Copyright (c) 1988-1993 The Regents of the University of California.
 * Copyright (c) 1994 Sun Microsystems, Inc.

Checking https://opensource.apple.com/source/tcl/tcl-14/tcl/license.terms, the TCL license appears to apply.

Both of these licenses require the copyright notice and the license text to be distributed in all copies. Please add the applicable full license text, either in separate files or in LICENSE.txt.

What versions are you using?

  • OS: N/A
  • Python: N/A
  • UltraJSON: 5.5.0

Please include code that reproduces the issue.

Not applicable.

@musicinmybrain
Copy link
Contributor Author

Actually, it looks like the MODP_ASCII code was included no later than 2012, so it would fall under the BSD-3-Clause license that was in effect at the time rather than the current MIT license. Similar requirements for distribution of license text still apply.

@hugovk hugovk added the documentation Documentation label Sep 26, 2022
@hugovk
Copy link
Member

hugovk commented Sep 26, 2022

Thanks for raising this, would you like to create a PR?

musicinmybrain added a commit to musicinmybrain/ultrajson that referenced this issue Mar 10, 2023
Both of these licenses require the copyright notice and the license text
to be distributed in all copies. Instead of merely mentioning them,
include the full text in LICENSE.txt

Fixes ultrajson#565; see that issue for further analysis.
musicinmybrain added a commit to musicinmybrain/ultrajson that referenced this issue Mar 10, 2023
Both of these licenses require the copyright notice and the license text
to be distributed in all copies. Instead of merely mentioning them,
include the full text in LICENSE.txt

Fixes ultrajson#565; see that issue for further analysis.
@musicinmybrain
Copy link
Contributor Author

Thanks for raising this, would you like to create a PR?

I got around to it eventually!

bwoodsend pushed a commit that referenced this issue Mar 10, 2023
Both of these licenses require the copyright notice and the license text
to be distributed in all copies. Instead of merely mentioning them,
include the full text in LICENSE.txt

Fixes #565; see that issue for further analysis.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants