On 08/21/2015 05:10 PM, Pavel Reichl wrote:

Petr can you change data_provider_req.c to include

"providers/data_provider_req.h"

instead of

"providers/data_provider.h" ?

I originally thought that you will be able to include solely data_provider_req.h from responder_dp.c but I see that data_provider.h is required. But I still don't mind introducing data_provider_req.h.

If you change this little nitpick I think I can ACK the patch.

Pavel,
you're right, that's mine main opinion to this issue. I am sorry, I need more focus and coffe.
There is fixed (not repaired) patch.
Petr

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