On Mon, 2012-07-02 at 12:56 -0400, Stephen Gallagher wrote: > > Fixed (as mentioned above, I tied everything to > ad_get_common_options()) > > > New patches since review: > > Patch 0008 is the same as sent earlier, Patch 0009 adds the AD > provider > into the contrib spec (forgotten earlier). >
Manual code review looks fine now. Although I haven't actually tested the code myself I would give an ACK here. Simo. -- Simo Sorce * Red Hat, Inc * New York _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel