-
Notifications
You must be signed in to change notification settings - Fork 12
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
Licensing clarification #217
Comments
Definitely sounds updating headers to consolidate licensing is needs. Admittedly most files were left untouched after we took over the project from the original maintainers. @leandro-lucarella-sociomantic, thoughts on this? |
Not really, I wouldn't touch anything to avoid problems. I guess if you need to redistribute in binary form, just include all the copyrights. It is sort of normal for different files to have different copyrights. |
However I guess I should change the copyright with my name to:
|
Add it to the files we touched. Yes. |
Could we make COPYING and docs/license.rst have the same contents (or point one to the other), and make its contents the union of all the other licenses/copyrights? Initially, I had assumed that obeying the top-level COPYING file would be sufficient to legally distribute this library. (Otherwise, why would that file exist, if it's not accurately describing how I can copy it?) It's only by accident that I discovered I'd be violating 10+ other licenses. I bet nobody using this library is giving credit properly, because it's not easy to see what to do. I'm not suggesting spending any effort on changing any licenses. I'm simply proposing that it's a worthy goal for an open-source project to make it easy for people using the project to do the right thing, e.g., by putting all the license information together in one place.
It does happen, but in every case I've seen, the main license file either contains all the licenses, or links to them. For example, on plcrashreporter, the license file has the primary license, and then sections below that for:
Even though there's 2 licenses and 4 copyrights, it's easy to understand just from looking at the LICENSE file. |
Another thing I just noticed: every file is has a "BSD" license, but it's not always the same. Most have a standard-looking 3-clause BSD license, with the third clause that reads:
but in one of them, the third clause was changed (in da84cc1) to read the opposite:
I can't tell if this was intentional or not. |
The issue here is we are not the original authors, we just picked an abandoned library because we needed to use it and we thought it was nice to keep our modifications public in case someone else find them useful. So taking a look at exactly how many licenses are in there and find a way to put all in one file would be a huge task for us as for any other person. Since we don't distribute this library in binary form, it's very hard to justify this investment for us. I guess the original authors should be contacted, etc. We are more than happy to accept PRs to improve this situation, but I don't see us working on this any time soon. |
Will close the issue but it might be reopened if we decide to simplify the licensing. |
Why closing it? It is in fact still an (open) issue and an important one. I would leave it open so is visible to potential contributors that could fix it. |
Reopening |
This project is under a BSD license, but beyond that, the license situation is not clear to me. The top-level COPYING file says:
while the docs/license.rst file says:
Additionally, each source file seems to have a different copyright in its header comment. For example, libdrizzle/pack.cc is:
while libdrizzle/poll.cc is:
and libdrizzle/structs.h is:
In all, there are 9 different BSD copyrights in the various source files, and each one says "Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer".
Is it correct, then, that I need to include 11 different BSD licenses, one for each of the various copyrights, in order to distribute a binary containing libdrizzle?
IANAL but perhaps a way could be found to consolidate these.
The text was updated successfully, but these errors were encountered: