search

conda-forge / python-ldap-feedstock

A conda-smithy repository for python-ldap.
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Unnecessary dependency on `openldap` #4

Open p-himik opened 4 years ago

p-himik commented 4 years ago

Issue: openldap is specified as a dependency, but the package itself does not depend on it in any way. A common use-case where openldap is not needed is when an LDAP instance is deployed to some other server.

bkreider commented 3 years ago

I'm not sure this is true. I tried to build using my own recipe and it failed looking for openldap libraries.

p-himik commented 3 years ago

It's been more than a year, so I can't really recall right now why was the root cause for me creating the issue. I regret not putting enough details in the original message.

Right now I can see that python-ldap definitely depends on libldap, and probably always have been. And openldap does provide libldap. However, openldap provides much more than that. Today it seems not to be that big of a deal because it's just a few more MBs, but either I was much more stringent a year ago or openldap was taking much more space and/or was taking a long time to install.

Ideally, there should be a separate conda package for just libldap. It's probably out of the scope of this project, however this project might be a perfect use-case for creating such a package. I'll let the maintainers decide whether to close this issue or not.

bkreider commented 3 years ago

That makes sense. I agree with you. libldap could be split out as a conda package. Maybe that should be an issue in the openldap recipe and this one would depend on that being resolved?

p-himik commented 3 years ago

Makes sense! Just created it.