web icon indicating copy to clipboard operation
web copied to clipboard

[16.0][IMP] web_widget_dropdown_dynamic: Change license

Open ivs-cetmix opened this issue 10 months ago • 7 comments

Change license to LGPL-3 in order to be able to use this module with other modules with non-AGPL-3 licenses.

@alexey-pelykh @NL66278 @BankyKmitl

ivs-cetmix avatar Jan 08 '25 17:01 ivs-cetmix

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar Feb 26 '25 15:02 OCA-git-bot

I don't like to open the door to private extensions on top of this one, but it's not my choice in this case as I'm not a contributor of this module.

You have to change though all the copyright texts in the file headers if the switch is done.

pedrobaeza avatar Feb 26 '25 16:02 pedrobaeza

I don't like to open the door to private extensions on top of this one, but it's not my choice in this case as I'm not a contributor of this module.

You have to change though all the copyright texts in the file headers if the switch is done.

May I ask you why exactly? I prefer the AGPL license as well, however in some cases it creates havoc while giving no benefit to community. I would personally prefer to have things like widgets and tools being LGPL, while core industry modules stay AGPL. This is because with LGPL we can increase potential auditory and benefit from the widespread of the OCA solutions making them a crucial part of other tools. Exactly the way it happened with queue_job

ivs-cetmix avatar Feb 26 '25 16:02 ivs-cetmix

This is a backend tool, so I don't want my developments that I'm giving for free are used for building other things that are not shared. I don't mind the audience. I mind that everything stays open, so if you don't want to follow that rules, be on your own. At the end, I have seen that I have to maintain the same modules being AGPL / LGPL, so I don't benefit on any hypothetical audience increase.

I just make the exception on website ones, where being accessible to the public, and with customized layers like themes and so, I don't want that someone may require the custom/theme code to my customer.

pedrobaeza avatar Feb 26 '25 17:02 pedrobaeza

This is a backend tool, so I don't want my developments that I'm giving for free are used for building other things that are not shared. I don't mind the audience. I mind that everything stays open, so if you don't want to follow that rules, be on your own. At the end, I have seen that I have to maintain the same modules being AGPL / LGPL, so I don't benefit on any hypothetical audience increase.

I just make the exception on website ones, where being accessible to the public, and with customized layers like themes and so, I don't want that someone may require the custom/theme code to my customer.

I understand and share your concerns. However I would mention once again: this is just a widget. This module has zero business logic in it. Keeping it AGPL will force to develop a slightly modified copy of it under LGPL license and will end up with two similar widgets. Regarding the benefit of hypothetical audience increase, this is something that should be address at the Organisation level and is more related to marketing rather than to technical part. Recent changes raise some hopes that this can be the case.

P.S. as @Bearnard21 mentioned above, we have changed license in our module because @rvalyi asked us about it. P.S.S. there were already license changes in modules in this repo.

ivs-cetmix avatar Feb 26 '25 17:02 ivs-cetmix

P.P.P.S updated license in code too.

ivs-cetmix avatar Feb 26 '25 17:02 ivs-cetmix

Hi @rvalyi would appreciate your review here too.

ivs-cetmix avatar Mar 16 '25 13:03 ivs-cetmix

Dear @OCA/web-maintainers would appreciate your merging and reviewing it as it is "ready to merge". Thank you in advance!

ivs-cetmix avatar May 15 '25 18:05 ivs-cetmix

@ivs-cetmix I share the same view on licensing. However you should get the agreement from CroporateHub the author here and from the contributors eventually.

Theoretically, the OCA board would have the power to pass over, but that would create a precendent that would undermine the trust of module authors. So you should go after the module authors and get them agree here...

rvalyi avatar May 15 '25 18:05 rvalyi

@ivs-cetmix I share the same view on licensing. However you should get the agreement from CroporateHub the author here and from the contributors eventually.

Theoretically, the OCA board would have the power to pass over, but that would create a precendent that would undermine the trust of module authors. So you should go after the module authors and get them agree here...

@rvalyi CroporateHub has already approved this PR as @alexey-pelykh is the CEO and the owner of it. So I don't see any legal obstacles to move forward.

ivs-cetmix avatar May 15 '25 18:05 ivs-cetmix

Hi all. Two remarks here.

@Pedro said :

but it's not my choice in this case as I'm not a contributor of this module.

@rvalyi said :

However you should get the agreement from CroporateHub the author here and from the contributors eventually.

I don't think so. As far as I understand, when we contribute modules to the OCA, we do it under a CLA. As soon as the code is accepted, the entity that can change the licence of the OCA module is not CroporateHub, but OCA. Or did I missed something regarding the CLA ? Ref : https://raw.githubusercontent.com/OCA/odoo-community.org/master/the-association/ICLA.pdf Thanks for your insight here, because maybe I misunderstood some legal points here.

@ivs-cetmix said :

Keeping it AGPL will force to develop a slightly modified copy of it under LGPL license and will end up with two similar widgets.

Well, you have another option : you can put an AGPL licence on the module you develop, that depends on web_widget_dropdown_dynamic. ;-) In fact, that is exactly the objective of this licence.

That giving said, I abstain on that PR.

legalsylvain avatar May 15 '25 20:05 legalsylvain

Croporate

🙄 :) Corporate

alexey-pelykh avatar May 15 '25 20:05 alexey-pelykh

Ok, we have a board meeting next week. Will raise this question. Thank you for your feedback!

ivs-cetmix avatar May 16 '25 07:05 ivs-cetmix

@simahawk done, could you please check and merge?

ivs-cetmix avatar May 22 '25 12:05 ivs-cetmix

@legalsylvain as far as I know, the module author retain the copyright but also delegates it to the OCA board. That is technically both can decide.

But if the OCA starts taking decisions that goes against modules authors we have to understand the deep implications that would have: that would de facto totally undermine the trust contributors have in the OCA entity and contributors will simply assume the OCA will be the new OpenERP, ready to screw everybody as it is more convenient for itself.

But if the module author simply disappeared, of course the situation is different than an author firmly opposed to a license change...

rvalyi avatar May 22 '25 17:05 rvalyi

@rvalyi the author (me) approves and ~couldn't care less~ does not have a preference between the original and suggested license for this module :)

alexey-pelykh avatar May 22 '25 17:05 alexey-pelykh

@rvalyi the author (me) approves and ~couldn't care less~ does not have a preference between the original and suggested license for this module :)

great! thank you for showing up.

rvalyi avatar May 22 '25 17:05 rvalyi

actually @alexey-pelykh it seems you coded the module while employed by CorporateHub which is the declared module author and by checking your Github it seems you are not part of this organization or not anymore.

Let's just remember that technically in general when employees do some work their employer or eventually customer retain the copyright.

So @alexey-pelykh does your agreement here for a license change legally represents CorporateHub? How do we know? That would fully protect the OCA from a potential shitstorm...

Now, my personal opinion here is as technically the OCA board also holds the copyright and as at least the person who coded the module agrees explicitly on the change, then that would already aliviate a lot the "moral" responsability of the OCA in doing the change. That's why I approved the PR.

rvalyi avatar May 22 '25 18:05 rvalyi

@rvalyi

https://github.com/orgs/corporatehub/people - switched to public https://euipo.europa.eu/eSearch/#details/trademarks/018787771

alexey-pelykh avatar May 22 '25 18:05 alexey-pelykh

great! thank you for showing up.

@rvalyi also, just in case, I was the very first one who approved this PR same day it was created.

P.S. OCA's speed of reaction is as always unmatched 🤣

alexey-pelykh avatar May 22 '25 18:05 alexey-pelykh

great! thank you for showing up.

@rvalyi also, just in case, I was the very first one who approved this PR same day it was created.

P.S. OCA's speed of reaction is as always unmatched 🤣

Ok thank you. Well, please simply understand that your early approval "I'm fine with this change" had nothing to help us to relate you to CorporateHub which is the module author and copyright holder in the files. Some people may know it's the company you are working for, but I'm afraid you cannot take it for granted...

But okay anyway this is solved now I think. cc @ivs-cetmix @pedrobaeza @legalsylvain

rvalyi avatar May 22 '25 18:05 rvalyi

Now, my personal opinion here is as technically the OCA board also holds the copyright and as at least the person who coded the module agrees explicitly on the change, then that would already aliviate a lot the "moral" responsability of the OCA in doing the change. That's why I approved the PR.

As for this, depends on presence of explicit clause in CLA about pass of copyright. And that would mean that only OCA should be indicated as copyright holder. And I'm not sure that's the case. So I guess you need approval from everyone who [owns copyright on the] contributed code to the module and thus - holds copyright to portions of work.

alexey-pelykh avatar May 22 '25 18:05 alexey-pelykh

Well here I think we fall into the "moral responsibility" of the OCA not to go against contributors. But IMHO minor contributors who didn't even got a copyright mention are not legit to oppose such a change. Again, legally speaking the OCA can do the change. I'm just here giving my opinion about the "moral responsibility" when doing so.

rvalyi avatar May 22 '25 18:05 rvalyi

👍 for asking the legitimacy of approval - that question must be asked :)

alexey-pelykh avatar May 22 '25 18:05 alexey-pelykh

/ocabot merge major

simahawk avatar May 23 '25 06:05 simahawk

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-3053-by-simahawk-bump-major, awaiting test results.

OCA-git-bot avatar May 23 '25 06:05 OCA-git-bot

Congratulations, your PR was merged at bad0707cbc8a08bec3608718ca7a91c55e63bb58. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar May 23 '25 06:05 OCA-git-bot