<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>