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

Very likely several copyright violations under t/text/ (non-free song texts) #655

Closed
xtaran opened this issue Dec 1, 2017 · 13 comments
Closed
Assignees

Comments

@xtaran
Copy link

xtaran commented Dec 1, 2017

I don't think the following song texts are published under the Artistic License 2.0 or any other free software or free artwork license:

This means that e.g. Debian needs to unpack your tar ball, remove those files, create a new tar ball for uploading to Debian as well as to patch the test suite to either skip the related tests or modify them to use other text files instead. And I'm sure, CPAN won't be happy about unlicensed lyrics either.

So I strongly recommend to either document the license of these song texts (if you have one) or remove them from the repository. (IMHO git rm should suffice, but IANAL.)

And JFTR: I report this, because Debian can't ship such non-free files. And IMHO it would be way better to remove those files from the original ack source distribution than just from the Debian package since then all ack users would and distributors would benefit from a clear legal situation and not only Debian, its derivatives and their distributors.

@petdance
Copy link
Collaborator

petdance commented Dec 2, 2017

@petdance
Copy link
Collaborator

petdance commented Dec 2, 2017

This means that e.g. Debian needs to unpack your tar ball, remove those files, create a new tar ball for uploading to Debian

@xtaran The way I'm reading https://packages.debian.org/stretch/all/ack/filelist, none of the test files are in the Debian package. Is that correct? Or am I looking at the wrong thing?

@gregoa
Copy link

gregoa commented Dec 2, 2017

They are in the source package (which is the original tarball plus the tarred up debian/ directory), and Debian also publishes these source packages, so they also have to consist only of files with free licenses.

If you e.g. look at https://packages.debian.org/source/testing/ack you'll see the ack_2.18.orig.tar.gz at the bottom, which is the same as https://cpan.metacpan.org/authors/id/P/PE/PETDANCE/ack-2.18.tar.gz.
Or on each Debian mirror, e.g. http://ftp.ch.debian.org/debian/pool/main/a/ack/ (here you see both the binary packages - the .debs whose contents you found at https://packages.debian.org/stretch/all/ack/filelist - and the source packages - .orig.tar.gz, .debian.tar.xz, .dsc).

@petdance
Copy link
Collaborator

petdance commented Dec 2, 2017

Thank you. I've never looked into any of the mechanics of Debian/Redhat/etc packaging.

they also have to consist only of files with free licenses.

I assume that things licensed under Creative Commons would be OK?

@gregoa
Copy link

gregoa commented Dec 2, 2017 via email

@petdance
Copy link
Collaborator

petdance commented Dec 2, 2017

exceptions are all -NC (non-commercial) and -ND (no derivatives) variants

Interesting. Thanks for that clarification.

@xtaran
Copy link
Author

xtaran commented Dec 3, 2017

@petdance: Thanks for caring!
@gregoa: Thanks for helping out while I was travelling.

@petdance
Copy link
Collaborator

petdance commented Dec 8, 2017

I've released a dev release 2.19_01 to CPAN, and it's tagged with 2.19_01 in git. Would one of you be so kind as to check it out, to make sure it all looks good?

I'd like to release an actual release, 2.20, in the next day or two.

@xtaran
Copy link
Author

xtaran commented Dec 8, 2017

Thanks! Will have a look this evening.

@petdance petdance self-assigned this Dec 8, 2017
@petdance petdance added this to the 2.20 milestone Dec 8, 2017
@xtaran
Copy link
Author

xtaran commented Dec 9, 2017

Thanks a lot for fixing this so quickly despite it surely was quite some work to replace all the excerpts in the tests!

Anyway, looks all fine for me:

  • The new replacement texts are all clearly identifiable as in the public domain (e.g. on Wikipedia).
  • The test suite passes both, at build-time as well as with as-installed testing (at least on my laptop running Debian Unstable, i.e. I can't verify if it fixes the Ubuntu as-installed test suite failures discussed in Ubuntu Artful doesn't contain ack (nor ack-grep) #652)

So from my point of view, 2.19_01 fixed this issue with copyrighted examples texts very well. Feel free to close this issue (#655).

Will upload ack 2.19_01 to Debian Experimental now (as it's a release candidate primarily meant for testing) and later the final release ack 2.20 to Debian Unstable.

@xtaran
Copy link
Author

xtaran commented Dec 9, 2017

To make it easier to check if this upload makes any change with regards to the autopkgtest issue in Ubuntu (#652) I changed my plan and uploaded 2.19_01 directly to Debian Unstable. Should hit Ubuntu Bionic (current development release) within this weekend.

@petdance
Copy link
Collaborator

petdance commented Dec 9, 2017

Thanks for checking into it. I have some problems on Windows with the tests that I may fix up before 2.20. But 2.20 should go out by Monday.

@petdance
Copy link
Collaborator

ack 2.20 has been released to CPAN and is on the master branch here. All texts in the t/text directory are now public domain.

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