On Mon, Jun 02, 2014 at 03:15:33PM +0200, Jakub Hrozek wrote:
On Fri, May 30, 2014 at 02:52:50PM +0200, Pavel Reichl wrote:
> Thank you Jakub for comments. I proposed new version of text to man
> pages. I updated 2nd patch and generally speaking returned to solution
> which I proposed in 1st or 2nd iteration of this patches. But this time
> I traded off code beauty for performance, but I think it's OK.
>
> Regards,
>
> PR
ACK.
I tested the patch with LDAP provider connected to an IPA server, AD
provider connected to an AD server and finally with LDAP provider
connected to an AD server. The first two worked as I would have
expected.
The third one, however did not, seems like we broke that combination in
1.11:
https://fedorahosted.org/sssd/ticket/2345
But that issue is not related to your patches, so we can move forward
and fix that bug separately.
Pushed to master:
69994add9cd4e57d40b3b7a0b1783ef2d0aa974c
c30f1d05d6c75080fbe8ea9868f291e7a331ad44
3c1899348804713b49ba9c1f2bc782892c47c2fa
and to sssd-1-11:
4221bd76e2b631684f2dc7e8c625fd7b27947cf8
98052f6f186f27a6fde4786274132a6bb4d69e79
e97a7f5bfbb6de33fd722cf613dbaea042126ca6