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

Pavel Reichl preichl at redhat.com
Fri Aug 21 17:43:05 UTC 2015



On 08/21/2015 05:49 PM, Petr Cech wrote:
> 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 at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
ACK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20150821/270f63fa/attachment.html>


More information about the sssd-devel mailing list