On Tue, 2013-10-15 at 14:39 +0200, Lukas Slebodnik wrote: > 3. IIRC, sssd can be build with ldap without sasl support, but ipa > provider > will not work. And it should be similar with ad provider. I would > prefer > to use conditional build instead of failing if ldap does not > support sasl. > Because with your patch, sssd could not be build without sasl > library and > someone can use sssd only with ldap provider (and he doesn't care > about > sasl)
Lukas, I think it is fine to consider SASL a hard dependency for SSSD, although we shouldn't force it to be cyrus-sasl, it is unlikely openldap will move any time to soon to an alternative library, so just pretending cyrus-sasl is the only one we care about is fine. That said I agree just checking sasl is installed does not mean openldap has been built with it, so we need better detection for openldap. Simo. -- Simo Sorce * Red Hat, Inc * New York _______________________________________________ sssd-devel mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
