Added new license OpenLDAP 2.7. #66
Conversation
lib/Software/License/OpenLDAP_2_7.pm
Outdated
| sub name { 'The OpenLDAP Public License Version 2.7, 7 September 2001' } | ||
| sub url { 'http://www.openldap.org/doc/admin21/license.html' } | ||
| sub meta_name { 'openldap' } | ||
| sub meta2_name { 'openldap_2_7' } |
There was a problem hiding this comment.
meta_name and meta2_name must be from a fixed list of licenses: https://metacpan.org/pod/CPAN::Meta::Spec#license. For this license, I believe they should both be unrestricted.
There was a problem hiding this comment.
Hi @haarg
Thanks for your suggestions. I have pushed the changes.
Kindly review it again.
Many Thanks.
Best Regards,
Mohammad S Anwar
There was a problem hiding this comment.
Actually I believe it should open_source, not unrestricted, based on precedent (FreeBSD license in particular) and the description of meta1 keys in Module::Build
There was a problem hiding this comment.
The OpenLDAP license is obviously based on the 3-clause BSD license, but seems sufficiently different that I'm not sure it could be said to be OSI approved.
There was a problem hiding this comment.
The SPDX website lists it as OSI approved, but I can't find any proof of that on the OSI website. Who would have guessed licenses are a mess…
manwar
force-pushed
the
OpenLDAP-license
branch
from
0bdae5b to
cce6d01
Compare
|
The real solution is to add spdx support, I think we should merge that first before adding any other licenses. |
|
This is pointing to version 2.7 of the license, even though 2.8 has been out for over 15 years. Is there any particular reason for that? |
Hi Team,
Please review the PR.
It adds new license OpenLDAP. I came across a distribution on MetaCPAN that needs this license.
https://metacpan.org/release/Alien-LMDB
Many Thanks.
Best Regards,
Mohammad S Anwar