-
Notifications
You must be signed in to change notification settings - Fork 388
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
Move license to EPL 2 #683
Comments
I'll do this. It's about my speed. |
What about all the copyright comments in the java files? Also epl-1 is mentioned in a lot of about.html files. |
@SteveSchafer-Innovent please select one small bundle to apply EPL2.0 and let us discuss the required changes in details. Restoring all the chain of contributors could be problematic for every file, but, since we have git history, it is allowed to use formulation like |
Alexander, is point 3 really needed or should we file a new issue and solve that post 4.9? |
Wim, it should not be a big deal while it removes a lot of questions. Typically this is +several files per bundle, nothing notable comparing with amount of .java files. |
Ok. Let's see. The java files can be changed using a smart regex find/replace. Same for the about.html. |
My plan is to change all the files with scripting of one form or another. |
Maybe you can ask on cross project mailing list if someone already made these tooling. @vogella do you know.if there is eplv1-eplv2 tooling?
|
How do I access the cross project mailing list? |
I used regex expression for the main work and manual fixes then needed, see here: https://bugs.eclipse.org/bugs/show_bug.cgi?id=535802 |
There is this thing called google ;) |
I've created awk scripts for license.html, about.html, feature.properties, epl-2.0.html files, and java files. Should I delete the epl-v10.html files? Still searching for more instances of the license but I should be ready to submit a PR soon. |
Please, please, would you be so kind to not submit all the changes in one PR. |
I know, the changes are huge. EVERY java file is changed. Would you suggest one PR for each of the file types? |
I would prefer to have 1 PR per bundle |
If bundles are instances of plugin.xml then there are 234 of them. Is that what you mean? |
Can we start from 1 (one) PR for the first bundle in the list? For example we can start from what is inside this location https://github.com/eclipse/birt/tree/master/UI/org.eclipse.birt.report.debug.core |
That location doesn't contain epl-v10.html, feature.properties, or license.html, but it's a good test for about.html and java files. For the copyright notice in java files, I made the script smart enough to pull the creation year from the existing notice but otherwise I ignored everything else and replaced it with this:
I could make the script pull the original contributor name if you think that's necessary. Otherwise is this acceptable? Also, if the java file did not contain a notice, I added one with 2021 as the creation year. |
because this is not the feature location but bundle location. let's discuss features separately. Regarding suggested header, @waynebeaton we need your advice |
That header is fine as a go-forward solution, but do not remove existing copyright statements. For completeness, you can do things like add additional copyright holders or "and others" to an existing copyright statement. Or, you can add an entirely new copyright line to an existing header. FWIW, I was pretty quickly able to identify a header that contains copyright information that is not reflected in the commit log (i.e., the copyright owner was not actually the author or committer of a commit). |
If I do not remove existing copyright statements, then I leave the reference to EPL-1, correct? Or add a new copyright to existing header that mentions EPL-2? |
No. You can/should change the license statement. Just leave the copyright statements intact. |
Change all occurrences of Eclipse Public License version 1 to version 2 in UI/org.eclipse.birt.report.debug.core. Signed-off-by: Steve Schafer <sschafer@innoventsolutions.com>
PR for https://github.com/eclipse/birt/tree/master/UI/org.eclipse.birt.report.debug.core submitted. This project is fairly limited. I've found license references in more files in other projects, including *.html, *.exsd, *.msg, various .properties and .xml files, *.jj files, whatever those are. Some of the html files will be a bit more challenging to parse. I'll be committing the scripts to https://github.com/SteveSchafer-Innovent/Birt-Eclipse-License-Upgrade-Scripts. |
Suggested corrections. Signed-off-by: Steve Schafer <sschafer@innoventsolutions.com>
There are a number of license.properties files as well as *.msg files that contain a big text block that includes a reference to EPL 1. Is it sufficient to change the reference to EPL 2 or do other parts of that text need to be changed? Specifically there is a date near the top:
|
Yes, that needs to be changed to correctly reflect the EPL2 header. |
Changes to report/org.eclipse.birt.report.engine. This has a greater variety of file types than the previous PR. Signed-off-by: Steve Schafer <sschafer@innoventsolutions.com>
Change all occurrences of Eclipse Public License version 1 to version 2 in features/org.eclipse.birt.sdk/feature. Signed-off-by: Steve Schafer <sschafer@innoventsolutions.com>
I've submitted PR's for several projects, including a feature project. Are we ready to do the whole thing now? |
Yes, go ahead. |
Change all occurrences of Eclipse Public License version 1 to version 2 in the entire repository. 12391 files changed! Time to get some other eyeballs on this. Signed-off-by: Steve Schafer <sschafer@innoventsolutions.com>
Thanks, Steve! A hell of a job. Well done. |
The current license is EPL1 and we must move to EPL2.
We need to replace the following files:
license.html
epl-v10.html
and fix all texts in feature.xml and feature.properties.
See examples of the new files here:
https://github.com/eclipse/nebula/tree/master/widgets/oscilloscope/org.eclipse.nebula.widgets.oscilloscope.feature
The text was updated successfully, but these errors were encountered: