On Tue, 2012-01-31 at 16:02 +0100, Jakub Hrozek wrote: > We will want to set this option (but perhaps not > --enable-all-experimental-features) for F17 to be able to build the sudo > library. > > https://fedorahosted.org/sssd/ticket/1145 > > At the time being the option is also turned on when > --enable-all-experimental-features is specified. > > The second patch does the refactoring mentioned in #1145 - just moves > code around so that there are no #ifdefs in the main part of LDAP code.
Ack to the first, Nack to the second. There's no need for ldap_sudo.c AND sdap_sudo.c. The only reason we still have ldap_*.c in the code is legacy. We switched to the 'sdap' prefix a long while ago to avoid potential conflicts (as well as confusion whether a particular file was part of SSSD or openldap). Just put it into sdap_sudo.c please. (One of these days we should just rename those old files to avoid confusion...)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/sssd-devel
