On 08/28/2015 03:13 PM, Petr Cech wrote:
I am doing code review for fixed patch now. Patch looks good to me and
it works.

I discussed the problem above (about fprintf vs. libpopt) with both
offline. The libpopt solution starts here [1] and I agree it would not
be straightforward. So I am inclined to the solution presented in the
patch.

Regards

Petr

[1] src/tools/sss_override.c:1412

PS: I am waiting for CI tests.

CI: http://sssd-ci.duckdns.org/logs/job/23/84/summary.html
Failing is not connected to this patch.

=> ACK

Petr
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to