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
Setup Perl scripts still require perl-mozilla-ldap #2501
Comments
Comment from mreynolds (@mreynolds389) at 2017-11-06 19:14:28 We are planning on moving to a python installer for 1.4.0, but until that is done, regardless if "--with-openldap" is set, we must still use perLDAP (perl-Mozilla-LDAP) for these perl scripts. "--with-openldap" was designed to specify what client library the core server would use (mozilla or openldap). It was not intended to impact the scripts, or completely remove the "mozilla" dependency from the package. Anyway, I think the work on the new python installer is almost complete - it just has not been committed yet as we've all been busy wrapping up work on 1.3.7. That being said, for now we will probably still need perldap for other CLI tools like register-ds-admin.pl. We do want to get rid of all the perl (which would remove the perldap requirement) in 1.4.0, but that work has not really begun yet, so no promises. |
Comment from mreynolds (@mreynolds389) at 2017-11-06 19:14:29 Metadata Update from @mreynolds389:
|
Comment from imntreal at 2017-11-06 22:28:09 Sounds like I need to get back to trying to make mozldap build on Arch, again. I think it's been broken for a while. |
Comment from firstyear (@Firstyear) at 2017-11-06 23:05:26 @ImNtReal How soon do you need this packaged by? If you were willing, you could pass in "--disable-perl" to the ./configure, and try to use our python3 tools instead. The create/remove of the server is pretty solid today, and we can already run the majority of our test cases with them. If you are not in a rush to package this, it would be good for you to do this the "future" way, rather than spending a lot of time on this when we are just going to deprecate it so soon. |
Comment from imntreal at 2017-11-07 02:29:09 I would prefer using the new method, and helping test the "future" way. I'm already running 1.4.0.2. Apparently, between upgrading to 1.4.0.1, and .2, I got rid of the Mozilla LDAP packages, and now they won't build. The only functionality I need personally is the update functionality provided by the setup scripts. I haven't seen any mention of a straight forward way to do that with the python tools. Is there already a python script written that provides that function? |
Comment from firstyear (@Firstyear) at 2017-11-07 03:23:04 There have been no "update" scripts provided in a long time, and there is no plan to add them into the python. Instead the current design idea is to integrate this into server core so that upgrades happen at "startup" instead. This is so that it works correctly in containers. So there is no equivalent to setup-ds.pl -u, because it won't be needed. You only need the python tools for creating the instance and that's it. :) If you need more guidance I can write something up on how to use these. The current wiki pages are a little outdated so I should update this I think |
Comment from firstyear (@Firstyear) at 2018-01-30 00:16:08 Going to close this as we are about to release the python tools and there is no work for us to do here. These will be annouced on our 389-devel list and will be in 1.4.x. Thanks, |
Comment from firstyear (@Firstyear) at 2018-01-30 00:16:08 Metadata Update from @Firstyear:
|
Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/49442
Issue Description
The setup-ds.pl and setup-ds-admin.pl scripts still seem to require perl-mozilla-ldap even if built --with-openldap.
Package Version and Platform
389-ds-base 1.4.0.2-1 built from source on Arch Linux
Steps to reproduce
Actual results
See:
load perldap
use Mozilla::LDAP::Conn;
use Mozilla::LDAP::Utils qw(normalizeDN);
use Mozilla::LDAP::API qw(ldap_explode_dn);
use Mozilla::LDAP::LDIF;
Expected results
Not finding references to Mozilla:LDAP
The text was updated successfully, but these errors were encountered: