-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed
Description
The following files are licensed under GPL-3+:
plugins/flp_import/unrtf/*
Files are included in plugins/flp_import/unrtf.cpp. This commit changed the license: 98371cb
plugins/LadspaEffect/swh/vocoder_1337.c
This one is being handled in #2366.
plugins/zynaddsubfx/zynaddsubfx/src/Nio/AlsaEngine.cpp
plugins/zynaddsubfx/zynaddsubfx/src/Nio/AlsaEngine.h
plugins/zynaddsubfx/zynaddsubfx/src/Nio/JackEngine.cpp
plugins/zynaddsubfx/zynaddsubfx/src/Nio/JackEngine.h
These files are not enabled.
Activity
tresf commentedon May 3, 2016
Good find.
Albeit obvious, stating that we should switch them to GPL2 to match the source license would put more definition around this task (and then explain who we have to contact to do so, etc.)
jasp00 commentedon May 4, 2016
Regarding unrtf, the first to contact is Dave Davey daved@physiol.usyd.edu.au. It looks like LMMS is using version 0.20.5, from 2008 (ftp://ftp.gnu.org/pub/gnu/unrtf/). Then, there is David Santinoli without contact information.
softrabbit commentedon May 4, 2016
And 0.20.3 (7 months earlier) is the last GPL2 version. Looks little only 4 of the files had real changes between 0.20.3 and 0.20.5, so downgrading might be feasible, too.
jasp00 commentedon May 5, 2016
According to
plugins/zynaddsubfx/zynaddsubfx/README.txt, ZynAddSubFX is under GPL-2 only. In version 2.5.2, the four files listed above are under GPL-2 only. This would make LMMS undistributable as a whole; it could be distributed separately from the affected plugins.fundamental commentedon May 5, 2016
@jasp00 that file is out-of-date and the issue has been resolved upstream. I can dig up the mailing list post where the license was changed to GPLv2+ if you need me to.
jasp00 commentedon May 6, 2016
I see that version 2.5.4 is under GPLv2+. I am curious about the mailing list post; I have found this one from 2007, which makes little sense. But if upstream has gathered the necessary permissions, then it is all right.
fundamental commentedon May 6, 2016
That's the post. At that stage in the project's history all commits were done by that author.
jasp00 commentedon May 6, 2016
Why later versions were released under GPLv2 only? In Debian, there is this information about ZynAddSubFX:
(GPL-2 means GPLv2 only.) Did Lars Luthman, Ryan Billing and Harald Hvaal consent to the license change?
fundamental commentedon May 6, 2016
Debian had the wrong license. After the license change to GPLv2+ there was some confusion due to how the license was changed within individual files and as such debian mislabeled the project.
jonassmedegaard commentedon May 6, 2016
Ignore the Debian info, then - that's just a reflection of actual copyright and licensing statements withing sourcecode itself.
The following files, in the sourcecode of zynaddsubfx as redistributed by Debian, has GPL-2 (without "...or later") as license:
src/DSP/AnalogFilter.cpp
src/DSP/AnalogFilter.h
src/DSP/FFTwrapper.cpp
src/DSP/FFTwrapper.h
src/DSP/Filter.cpp
src/DSP/Filter.h
src/DSP/FormantFilter.cpp
src/DSP/FormantFilter.h
src/DSP/SVFilter.cpp
src/DSP/SVFilter.h
src/DSP/Unison.cpp
src/DSP/Unison.h
src/Effects/Alienwah.cpp
src/Effects/Alienwah.h
src/Effects/Chorus.cpp
src/Effects/Chorus.h
src/Effects/Distorsion.cpp
src/Effects/Distorsion.h
src/Effects/DynamicFilter.cpp
src/Effects/DynamicFilter.h
src/Effects/Echo.cpp
src/Effects/Echo.h
src/Effects/Effect.cpp
src/Effects/Effect.h
src/Effects/EffectLFO.cpp
src/Effects/EffectLFO.h
src/Effects/EffectMgr.cpp
src/Effects/EffectMgr.h
src/Effects/EQ.cpp
src/Effects/EQ.h
src/Effects/Phaser.cpp
src/Effects/Phaser.h
src/Effects/Reverb.cpp
src/Effects/Reverb.h
src/globals.cpp
src/globals.h
src/main.cpp
src/Misc/Bank.cpp
src/Misc/Bank.h
src/Misc/Config.cpp
src/Misc/Config.h
src/Misc/Control.h
src/Misc/LASHClient.cpp
src/Misc/LASHClient.h
src/Misc/Master.cpp
src/Misc/Master.h
src/Misc/Microtonal.cpp
src/Misc/Microtonal.h
src/Misc/Part.cpp
src/Misc/Part.h
src/Misc/Recorder.cpp
src/Misc/Recorder.h
src/Misc/Stereo.cpp
src/Misc/Stereo.h
src/Misc/Util.cpp
src/Misc/Util.h
src/Misc/WaveShapeSmps.cpp
src/Misc/WaveShapeSmps.h
src/Misc/WavFile.cpp
src/Misc/WavFile.h
src/Misc/XMLwrapper.cpp
src/Misc/XMLwrapper.h
src/Nio/AlsaEngine.cpp
src/Nio/AlsaEngine.h
src/Nio/AudioOut.cpp
src/Nio/AudioOut.h
src/Nio/Engine.cpp
src/Nio/Engine.h
src/Nio/JackEngine.cpp
src/Nio/JackEngine.h
src/Nio/JackMultiEngine.cpp
src/Nio/JackMultiEngine.h
src/Nio/MidiIn.cpp
src/Nio/MidiIn.h
src/Nio/NulEngine.cpp
src/Nio/NulEngine.h
src/Nio/OssEngine.cpp
src/Nio/OssEngine.h
src/Nio/OssMultiEngine.cpp
src/Nio/OssMultiEngine.h
src/Nio/PaEngine.cpp
src/Nio/PaEngine.h
src/Nio/WavEngine.cpp
src/Nio/WavEngine.h
src/Output/DSSIaudiooutput.cpp
src/Output/DSSIaudiooutput.h
src/Params/ADnoteParameters.cpp
src/Params/ADnoteParameters.h
src/Params/Controller.cpp
src/Params/Controller.h
src/Params/EnvelopeParams.cpp
src/Params/EnvelopeParams.h
src/Params/FilterParams.cpp
src/Params/FilterParams.h
src/Params/LFOParams.cpp
src/Params/LFOParams.h
src/Params/PADnoteParameters.cpp
src/Params/PADnoteParameters.h
src/Params/PresetsArray.cpp
src/Params/PresetsArray.h
src/Params/Presets.cpp
src/Params/Presets.h
src/Params/PresetsStore.cpp
src/Params/PresetsStore.h
src/Params/SUBnoteParameters.cpp
src/Params/SUBnoteParameters.h
src/Synth/ADnote.cpp
src/Synth/ADnote.h
src/Synth/Envelope.cpp
src/Synth/Envelope.h
src/Synth/LFO.cpp
src/Synth/LFO.h
src/Synth/OscilGen.cpp
src/Synth/OscilGen.h
src/Synth/PADnote.cpp
src/Synth/PADnote.h
src/Synth/Resonance.cpp
src/Synth/Resonance.h
src/Synth/SUBnote.cpp
src/Synth/SUBnote.h
src/Synth/SynthNote.h
src/Tests/AdNoteTest.h
src/Tests/AllocatorTest.h
src/Tests/ControllerTest.h
src/Tests/EchoTest.h
src/Tests/MessageTest.h
src/Tests/MicrotonalTest.h
src/Tests/MiddlewareTest.h
src/Tests/MqTest.h
src/Tests/OscilGenTest.h
src/Tests/PadNoteTest.h
src/Tests/PluginTest.h
src/Tests/RandTest.h
src/Tests/RtAllocTest.h
src/Tests/SubNoteTest.h
src/Tests/UnisonTest.h
src/Tests/XMLwrapperTest.h
src/UI/guimain.cpp
fundamental commentedon May 6, 2016
@jonassmedegaard Please notice the copyright and author fields associated with those files. This is nothing more than a collection of typos that originated at the original GPLv2->GPLv2+ change.
jonassmedegaard commentedon May 6, 2016
Quoting Mark McCurry (2016-05-06 04:05:40)
Please elaborate: What in particular should I notice about those
copyright holders, which makes the licenses of same files ignorable?
Jonas Smedegaard - idealist & Internet-arkitekt
Tlf.: +45 40843136 Website: http://dr.jones.dk/
[x] quote me freely [ ] ask before reusing [ ] keep private
fundamental commentedon May 6, 2016
@jonassmedegaard The majority of those files have the copyright and author field assigned to Nasca Octavian Paul, i.e. the individual who performed the relicensing to GPLv2+ in 2007. This issue has been resolved upstream.
jonassmedegaard commentedon May 6, 2016
Quoting Mark McCurry (2016-05-06 14:18:34)
URLs supporting that, please.
Jonas Smedegaard - idealist & Internet-arkitekt
Tlf.: +45 40843136 Website: http://dr.jones.dk/
[x] quote me freely [ ] ask before reusing [ ] keep private
fundamental commentedon May 6, 2016
zynaddsubfx/zynaddsubfx@8a403ab
The license is GPLv2+.
Anything stating otherwise was an omission erroneously done at this commit, or a result of result of incorrect refactoring which occurred due to a transfer of maintainership years after this transition.
If you want to see the email to the public mailing list corresponding with this commit, please just look at the previous post in this issue's thread. At that point all license holders agreed on this change.
8 remaining items
tresf commentedon May 8, 2016
Then I vote we simply do that.
tresf commentedon May 8, 2016
davedatphysiol.usyd.edu.aunow forwards to an unrelated email address, so no luck at all contacting the authors of the FL-import plugin.tresf commentedon May 9, 2016
This statement is slightly incorrect, it's him, he's just not an LMMS author. How did this happen? It appears it was the cherry-picking of
unrtfcode which placed him in the codebase as an author. Theunrtfproject uses GPL3, so it was most likely the motivation of the change.What does this mean for LMMS? We can choose to distribute with a mixed license, strip out the plugin entirely, or try to use the upstream library. Here's Dave's email response...
grejppi commentedon May 9, 2016
I just tried importing
NewStuff.flpfrom FL Studio 12.The result looks nothing like it should.
It is fine for projects made with old enough versions of FL Studio, but as the format evolves at Image-Line's whim, this feature is going to be less and less useful in the future.
In fact, if you ask me, I would be in favour of removing the FLP import feature altogether. The fewer unmaintained incomplete features, the better.
(Same goes for sample tracks tbqh)tresf commentedon May 9, 2016
@grejppi thanks, terrific feedback, much appreciated. 👍 from me.
liushuyu commentedon May 9, 2016
Or we can replace the implementation with a new one.
BTW, It should be noticed that MuseScore really does a good job at importing non-native project files.
jasp00 commentedon May 11, 2016
Perhaps unrtf is only used for comments (
FLP_Text_CommentRTF) to set the project notes. That item could be removed or refactored to use the externalunrtfapplication.paulnasca commentedon May 15, 2016
Hi.
I feel that there are needed some clarification from me regarding ZynAddSubFX.
Initially, in 2002, I have decided to release it under "GPL v2" and only under that version.
But, by request, I have changed the license of ZynAddSubFX to "GPL v2 or later" (including GPL v3, of course) in 2007 (more exactly, in 9 sep 2007, GIT commit on sourceforge 8a403abf9b80484efcd3253f3e04f5697f5742d0). I forgot to update license on some files by then, sorry, my bad. Please consider my part of ZynAddSubFX as being under "GPL version 2 or later", which includes GPL version 3+.
Thank you.