-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Copyright clarifications #4591
Comments
For reference, this is the included bug report:
|
The only two I can address off of the top of my head are the ICNS/Colorspace issue, which Debian has complained about before in other files, and the BMP test images. The colorspace issue is definitely something that Debian has referenced before, I'm not sure of the actual copyrightability of a standard colorspace that is describing an image, but Debian has felt strongly about it in the past. The BMP test images are from here: https://github.com/jsummers/bmpsuite. The images are public domain, except for the embedded icc profiles. The code to create them is gplv3. |
A quick bit of searching found this: https://metadata.ftp-master.debian.org/changelogs//main/t/tcl8.6/tcl8.6_8.6.6+dfsg-1_copyright Which is substantially the same license grant as the src/Tk/_tkmini.h file. |
https://github.com/python-pillow/pillow/blob/master/Tests/test_file_mcidas.py: # https://ghrc.nsstc.nasa.gov/hydro/details/cmx3g8
# https://ghrc.nsstc.nasa.gov/pub/fieldCampaigns/camex3/cmx3g8/browse/
test_file = "Tests/images/cmx3g8_wv_1998.260_0745_mcidas.ara" Added in #2552, which says:
|
https://github.com/python-pillow/Pillow/blob/master/Tests/fonts/LICENSE.txt says:
|
https://github.com/python-pillow/Pillow/blob/master/Tests/images/bmp/README.txt says:
|
DejaVuSans-bitmap.ttf, DejaVuSans.ttf, FreeMono.ttf, KhmerOSBattambang-Regular.ttf are missing from this list. DejaVuSans.ttf appears to be https://github.com/dejavu-fonts/dejavu-fonts/blob/master/LICENSE |
Fyi, I contacted libav.org about samples.libav.org, and got this reply
|
I received a more detailed answer from libav.org -
|
I've created PR #6062 to replace pillow3.icns with another version that does not have a copyrighted color space. |
Gathering together the responses in this issue.
That's an item for Debian, not for this repository.
As stated #4591 (comment), we have Tests/fonts/LICENSE.txt, and #5215 added missing fonts.
As stated in #4591 (comment), we have Tests/images/bmp/README.txt
#6062 resolved this by replacing the image.
In #4591 (comment) and #4591 (comment), I detail a response I got from libav.org about this.
As stated in #4591 (comment) / #2552, this is "free data from NASA".
My attempt to remove this difference in #5220 was declined. IANAL, but I see how it would be a problem for end users that some of our code has fewer rights.
#4591 (comment) seems to be saying that Debian already accepts code under a similar license. That leaves
nmake.opt became tiff.opt in #4084 and was then removed in #5359, but otherwise, these last two points still stand. So unless we wanted to chase down the authors of that code and ask for permission to fold their code into our regular Pillow license, has everything in this issue been addressed? |
That issue has been closed in April 2021 with "Fixed in version pillow/8.1.2+dfsg-0.1", so yes, let's close this too (and can reopen if needed). Thanks! |
During a review, Debian found some missing/unclear copyright issues in the pillow sources.
See https://bugs.debian.org/952899
For the Debian packages, I already remove Tests/icc and the testsuite gracefully skips these tests when this directory is not found, So I could extend that for other test cases as well. However I'd like to run as many tests as possible.
Could you have a look, what could be addressed upstream?
The text was updated successfully, but these errors were encountered: