Skip to content
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

License clarification #220

Open
tleguern opened this issue Jan 20, 2023 · 16 comments
Open

License clarification #220

tleguern opened this issue Jan 20, 2023 · 16 comments

Comments

@tleguern
Copy link

Hello,

It is unclear to me which license is used for the module keycloak-sync: the main LICENSE file says GPLv3 but keycloak-sync/src/setup.cfg says BSD, without precising which one.
Moreover the keycloak-sync/src/MANIFEST.in files explicitly includes the GPLv3 LICENSE.

Can you please clarify which of the two licenses apply and if it is BSD can you add the full license text?

Thank you.

@afabiani
Copy link
Member

It should be the same of GeoNode https://github.com/GeoNode/geonode/blob/master/LICENSE

@afabiani
Copy link
Member

@Zetten can you confirm?

@tleguern
Copy link
Author

It should be the same of GeoNode https://github.com/GeoNode/geonode/blob/master/LICENSE

GeoNode is GPLv2 which is also a different license.

@Zetten
Copy link
Contributor

Zetten commented Jan 23, 2023

@Zetten can you confirm?

The original author was @WernerRaath who may be able to shed more light on it.

I had assumed when making my (small) modification that it was covered under the LICENSE at the top of this repo. I didn't realise setup.cfg declared a different one.

@tleguern
Copy link
Author

Is this setup.cfg file automatically generated perhaps?

@afabiani
Copy link
Member

Yeps, I'm pretty sure the setup.cfg one was auto-generated, but still we need the author to decide wich license to apply.

@tleguern
Copy link
Author

Hello, any news on this subject?

@giohappy
Copy link
Contributor

This repo will need to be discussed by the GeoNode PSC since it's a bucket of community modules, some of them not maintained anymore, and without a clear attribution.

We should probably align its license to GeoNode (GPL 2) and ask authors to align their code to it. However, I don't think we will receive answers.

In any case, I would be in favor of archiving this repository and moving actively maintained modules to their own repositories.

@afabiani @gannebamm @t-book @francbartoli and anyone else, opinions?

@ridoo
Copy link

ridoo commented Jun 28, 2023

I did not contributed to this repository, but we wrote some GeoNode modules (mostly for 3.x and 4.x). Here's my view:

It is nice to have a location to explore written modules to get the idea "how things are done". This is for the good and the bad at the same time, thinking of copying/pasting things which are hacky at some point .

However, the main drawback I see is the maintenance issue of very different modules:

  • unclear maintenance responsibility
  • unclear what GeoNode version is actually supported
  • some are written for ancient GeoNode versions, but stand on the same level as newly written ones

The repository looks quite promising at a first glance, but does not hold on a second look, though.

I would tend to maintain a contrib module in a dedicated repository. In the end, branch management is left to the maintainer, but would better allow to organize support for different GeoNode versions over time.

"Good", or "well-known" contrib modules can be collected on a dedicated page nevertheless. When publishing a module to pypi.org you are able to tag it, too, so they can be found in context of a GeoNode search.

@giohappy
Copy link
Contributor

@ridoo yeah, I agree.

Also, when you want to install a module from this repo you get all or nothing, which is not good in many ways of course.
Good idea to have a page for modules on geonode.org (which, by the way, needs some love!).
Any contribution is well accepted of course.

@gannebamm
Copy link

@giohappy We discussed this internally while creating contrib apps as extensions for our GeoNode instance. To have better branch management and maintainability, we followed your idea and created a separate repository for the extension. You can take a look at it here: https://github.com/GeoNodeUserGroup-DE/contrib_externalapplications
Feedback is welcome! My idea was to get a feeling for this and later create a GNIP and discuss it with the PSC. You were faster :D

I like the idea of a managed list on geonode.org with contrib modules. Just like the gallery https://geonode.org/gallery/ but with contrib modules.

@francbartoli
Copy link
Member

Hi @giohappy, +1 from my end. Anyway, I like the idea of advertising them on the website and possibly to have a repository that acts as an aggregator of maintained modules only.

@gannebamm
Copy link

As discussed in this thread, we created our own repos per contrib apps. Currently, two are listed here:

We would like to adopt at least the geonode_ldap parts of this repository. Are we allowed to do this? We would start to get it properly working with the current GeoNode stable version.

https://github.com/GeoNode/geonode-contribs/commits/master/ldap
this delivers the following contributors: @giohappy @afabiani @t-book @ricardogsilva

Are we allowed to proceed with the above?

@ricardogsilva
Copy link
Member

@gannebamm

With regard to your request about using geonode_ldap code, and as the original contributor of such code, I am absolutely OK with it - please take it and use it. I suspect the opinion of the other people mentioned to be the same, but obviously I cannot speak for others.

The fact that it may have not originally been published with a suitable license was an oversight, but the intention surely was too be inline with the licensing terms of GeoNode, and to enable the community to freely use it for whatever needs.

The only thing I would add, and I know I'm straddling off-topic here, is that, as mentioned in this code, perhaps we'd do good to make the geonode user groups implementation a bit more standard - then we could retire this geonode_ldap module, as it would not need to exist anymore. Just food for thought.

@giohappy
Copy link
Contributor

@gannebamm sorry for the very late reply. I guess you have created your own repos for the contrib modules of your interest.

Are you planning to move them back under the Geonode umbrella, and prepare a page to list / avertize them?

@gannebamm
Copy link

Hi @giohappy
We have investigated the contrib module and concluded that we do not need the group propagation of the module at all for our purpose. Therefore, we used plain django-ldap. You can take a look at it here:
Thuenen-GeoNode-Development/thuenen_atlas@22e416c

This will only grant access and create an LDAP account on GeoNode and do not create or match any LDAP groups to GeoNode groups.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants