<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<br>
<div class="moz-cite-prefix">On 08/21/2015 05:49 PM, Petr Cech
wrote:<br>
</div>
<blockquote cite="mid:55D74891.3070700@redhat.com" type="cite">On
08/21/2015 05:10 PM, Pavel Reichl wrote:
<br>
<blockquote type="cite">Petr can you change data_provider_req.c to
include
<br>
<br>
"providers/data_provider_req.h"
<br>
<br>
instead of
<br>
<br>
"providers/data_provider.h" ?
<br>
<br>
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.
<br>
<br>
If you change this little nitpick I think I can ACK the patch.
<br>
</blockquote>
Pavel,
<br>
you're right, that's mine main opinion to this issue. I am sorry,
I need more focus and coffe.
<br>
There is fixed (not repaired) patch.
<br>
Petr
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sssd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
</blockquote>
ACK<br>
</body>
</html>