-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Java source files with missing or non-standard copyright notices #1001
Comments
These are created by @Nehon so I believe we can update them to standard JME copyright. |
@pspeed42 now that jme core utilise Tweens, do you think it would be good idea to deprecate them from Lemur and put them in jme core ? |
I think they were changed after moving, weren't they? |
Yes, the following has changed: @pspeed42 any issues with above changes ? @pspeed42 , @stephengold if you feel this is out of scope of this issue, and it would be better to get asked in forum, please let me know. |
The discussion here seems appropriate to me. |
I'm not ready to completely fold Lemur's version into JME yet. It's fine if JME takes them as a fork but I don't necessarily agree with how they were changed but neither do I want to make a hard decision about that right now. "ContainsTweens" is not a name I'd have ever picked... nor would I have returned the raw array. Not sure what it's used for, though. |
Ok, I see. Thanks.
Yes, maybe "CompositeTween" or "BranchTween" would be a better name... anyway it's upto @Nehon I guess ;)
This is only place I found it is used.
Should we keep SimSilica copyrights on those 3 classes (Tween.java, Tweens.java, AbstractTween.java) and use JME standard copyrights for the rest ? |
I just messaged @Nehon via a couple channels to clarify his intent. |
If the SimSilica copyright were used on those three specific classes and JME copyrights on others, does that resolve this issue? It seems like @pspeed42 has effectively given this permission to use the classes while maintaining their current copyrights. I think if the communication continued here, this issue could be resolved quite quickly, based on the interaction from these users above. |
If it's an issue, you have my permission to re-license those files. Given that the licenses are otherwise compatible, I guess that is the smaller issue of this thread, though. |
PR #1274 addressed the special case of TextureGeneratorWood.java by deleting the file, since it had an incompatible license and we've lost touch with its author. I'll go ahead and act on Paul's permission to re-license. However, that will leave a large number of files still to be dealt with. |
re-assign Simsilica LLC copyrights with Paul's permission (issue #1001)
If someone claims authorship of a file and then commits it to our repository without a copyright notice, I think we can assume they intended to contribute it to out project. If we're no longer able to contact that person, perhaps we should assume the omission was an oversight. In that case, it seems appropriate to add an appropriate copyright notice and license. I'll start making a list. |
|
Of those 467 files, 37 contain "seinturier", 50 contain "nehon", and 152 contain "autogenerated". The autogenerated Java files should NOT have copyrights, in my opinion. |
jme3-android-examples was largely the work of @iwgeric |
Need permission from @normen to add the JME copyright and BSD license to the following 4 classes:
|
sure, all code I authored and committed to the jme project is released under the same BSD license as the engine, no need to count peas |
Thanks, @normen. Just being extra cautious! |
As far as I can remember we always mentioned to contributors that the BSD license was a requirement for code to be included in the engine so really there shouldn't be any code in there with issues. All code from those in the old core team has been contributed with that understanding, code where BSD was not possible hasn't been included. |
I hope you're right! |
@pspeed42: Would it be OK for me to replace your copyright notice on |
Yeah, I didn't realize it wasn't the standard one. |
Thanks. I'll adjust it. |
SerializerRegistrationsMessage.java transferred at 3671ec9 in "master" branch. |
com.jme3.anim.uti.Primitives ought to have an Apache license. |
The authors of these classes need to be contacted so that we can make sure they've actually contributed them to the jMonkeyEngine project. Then the source files should be updated to the standard JME copyright notice.
checkers.quals.*
com.jme3.anim.AnimClip
com.jme3.anim.AnimComposer
com.jme3.anim.AnimTrack
com.jme3.anim.AnimationMask
com.jme3.anim.Armature
com.jme3.anim.ArmatureMask
com.jme3.anim.Joint
com.jme3.anim.MatrixJointModelTransform
com.jme3.anim.MorphControl
com.jme3.anim.SeparateJointModelTransform
com.jme3.anim.Weights
com.jme3.anim.interpolator.*
com.jme3.anim.tween.*
com.jme3.anim.tween.action.*
com.jme3.anim.util.*
com.jme3.app.DetailedProfiler
com.jme3.app.DetailedProfilerState
com.jme3.audio.AudioSource
com.jme3.audio.openal.AL
etcetera
The text was updated successfully, but these errors were encountered: