-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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
Joyent Copyright still in header of many files #3926
Comments
Those can probably go, but I'm not a lawyer. Also, see 3e1b1dd |
see also #527 / 767ee73 and /cc @bnoordhuis for further context |
So it would appear there are a handful of copyrights still active in the code base (not just dependencies). Owners of these include
It would appear the joyent copyright is present in LICENSE, which seems to be a central place for keeping project licenses There is also a stray license file in tools/doc --> https://github.com/nodejs/node/blob/master/tools/doc/LICENSE |
You have my written permission to remove any of my copyrights, just CC me in a Pull Request. |
We'll need a legal opinion on this prior to altering it. I can bring it to the next board meeting if the CTC requests it. |
Yes please. |
Getting an official legal position is good but in general, no existing copyright statement should be altered without the consent of the copyright holder or such alteration is acceptable within the terms of the license. For instance, when io.js originally removed the copyright statements from the individual files and into the main LICENSE file, that was ok because the terms of open source license were still being upheld. |
To re-iterate what @jasnell is saying: we can't do anything that materially changes or represents a material change of prior copyright assignment. If we changed this all to say "all prior code is owned by Mikeal Rogers" that would not actually assign anything to me, copyright law doesn't allow us to unilaterally re-assign someone else's copyright, and for compliance purposes we can't state something about the state of the copyright that is not accurate. What do we want to change this text to? |
I think we just want to consolidate it. |
BTW, we may want to start using some compliance tools to figure out where we have gaps in the stated licenses http://www.fossology.org/projects/fossology |
exactly what @Fishrock123 said. Things seemed inconsistent to me, so I figured it would make sense to have everything living in one place assuming that is allowed with the license |
Consolidation where possible is fine. Just keep in mind that some
|
I'm fine with removing the copyright boilerplate in
I'm not sure why I added that commit to that PR but the commit log suggests they were leftovers from 3e1b1dd. Probably a regex that wasn't rigorous enough. |
removing |
Moved to nodejs/TSC#174 |
Joyent copyright is still present in some headers of files in tools, test, support, and lib
Is this intentional? I'll follow up with a quick PR to fix this
The text was updated successfully, but these errors were encountered: