-
-
Notifications
You must be signed in to change notification settings - Fork 2k
[UPD][14.0] web_widget_x2many_2d_matrix: license to LGPL #3182
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
base: 14.0
Are you sure you want to change the base?
Conversation
@thomaspaulb There is also a module called quality_mrp_oca. Did not study this in detail, but could it not be a plugin compatible module for quality_mrp? |
That will have to involve them switching fully from Enterprise quality suite to OCA quality suite. I'm not ready to bear the burden of that currently |
I think you should also adapt the headers in the files. @pedrobaeza you approved the PR but your comment kind of mean you would disagree. Did you approved by mistake? Did I miss something? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally won't relicense any of my backend modules to LGPL. Anyway, I would say that the license incompatibility is the other direction, isn't it? @dreispt can say about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To increase adoption and attract more contributors, we propose changing the license of this module to LGPL, with consent from all authors.
LGPL reduces barriers for companies and SaaS providers, without compromising the spirit of open collaboration.
It’s a simple step to keep this module alive, useful, and growing.
Although the OCA can use the CLA to change a license (more precisely, can sublicense to an OSI-approved license), we don't want to do that against the will of the original authors. Note that later contributions, migrations, refactors, etc. do not grant authorship. Looking at the history of this module, it seems that the original author is @hbrunn under Therp BV. At this point, we would need this PR to be approved by a Therp representative. |
This is correct. |
The issue is actually with I did find a "goat's path" though: we can make a module that depends only on |
Me being the Therp representative, I could decide now then to merge this. I'm personally leaning towards the license change because this matrix module is meant as a tool to be widely used in all kinds of dependent modules. Though it could also mean that it's going to be used as part of third party closed source modules, on the app store. Tough choice. @hbrunn what's your opinion on the matter? |
Yes, indeed Tom. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update license in files as well. Eg here
@thomaspaulb I typically neither approve nor block this kind of license change, so do what you feel you need to be doing |
Funny, I've been having discussions about this in the board before, and now it happens to me for the first time, that I'm aware of.
Issue I have:
web_widget_x2many_2d_matrix
. Because the matrix module was AGPL, all dependencies are, and rightfully should be.quality_mrp
module.quality_mrp
does not play well with the matrix, so I need to make a glue module, which I think I'm not allowed to do, because it would depend both on an AGPL module and an Enterprise module, which have incompatible licenses.Do I have any other choice than this?