[SSSD] [PATCH] DATA_PROVIDER: BE_REQ as string in log message

Petr Cech pcech at redhat.com
Fri Aug 21 15:49:37 UTC 2015


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-DATA_PROVIDER-BE_REQ-as-string-in-log-message.patch
Type: text/x-patch
Size: 10175 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20150821/dcb5a23b/attachment.bin>


More information about the sssd-devel mailing list