<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">2013/10/7 Sumit Bose <span dir="ltr">&lt;<a href="mailto:sbose@redhat.com" target="_blank">sbose@redhat.com</a>&gt;</span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Oct 03, 2013 at 11:57:07PM +0200, Benjamin Franzke wrote:<br>
&gt; &gt;<br>
&gt; &gt; Hi Ben,<br>
&gt; &gt;<br>
&gt; &gt; this is really great, thank you very much for the contribution! As a<br>
&gt; &gt; matter of fact, we&#39;ve been planning on delivering the CIFS plugin as<br>
&gt; &gt; part of the next release:<br>
&gt; &gt; <a href="https://fedorahosted.org/sssd/ticket/1534" target="_blank">https://fedorahosted.org/sssd/ticket/1534</a><br>
&gt; &gt;<br>
&gt; &gt; Have you found the design page we created?<br>
&gt; &gt; <a href="https://fedorahosted.org/sssd/wiki/DesignDocs/IntegrateSSSDWithCIFSClient" target="_blank">https://fedorahosted.org/sssd/wiki/DesignDocs/IntegrateSSSDWithCIFSClient</a><br>
&gt;<br>
&gt;<br>
&gt; Yeah, I&#39;ve read those pages before writing the plugin..<br>
&gt;<br>
&gt;<br>
&gt; &gt;<br>
&gt; &gt; Mostly the work on our side was done by Sumit Bose (I&#39;ve CC-ed him). I<br>
&gt; &gt; would propose to get in touch with him to avoid duplicating some effort.<br>
&gt; &gt;<br>
&gt; Unfortunately Sumit is in Germany and there is a holiday so he won&#39;t be<br>
&gt; &gt; available until Monday.<br>
&gt; &gt;<br>
&gt;<br>
&gt; &gt; In general, I would suggest to merge this code into the SSSD tree if you<br>
&gt; &gt; wouldn&#39;t mind. I think it would makes sense to have the code in the same<br>
&gt; &gt; repo as the ID mapping library.<br>
&gt; &gt;<br>
&gt;<br>
&gt; I&#39;d really appreciate having it integrated in sssd tree,<br>
&gt; to have centralized maintainance and adoption by distributions..<br>
<br>
</div>Thanks a lot for the patch. I would suggest to put your code into<br>
src/lib/cifs_idmap_sss/ and modify SSSD&#39;s Makefile.am to build the<br>
plugin. Would you like to do the integration and send the result for<br>
review to this list again?<br></blockquote><div><br></div><div>I&#39;d like to do that integration, will send a patch..</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
About the well-know SIDs. They should not be handled inside the plugin<br>
but by SSSD itself. If you are interested in adding them I&#39;d be happy to<br>
help you how to start (if my help is needed at all :-).<br></blockquote><div><br></div><div>Yup, sounds better to handle the SIDs by SSSD core. I&#39;d like to try..</div><div>I&#39;ll see whether time permits me to do that soon.. (doing this as a part of my thesis)</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
bye,<br>
Sumit<br>
<div class="HOEnZb"><div class="h5"><br>
&gt;<br>
&gt;<br>
&gt; &gt; Thanks again for the contribution, much appreciated!<br>
&gt; &gt; _______________________________________________<br>
&gt; &gt; sssd-devel mailing list<br>
&gt; &gt; <a href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a><br>
&gt; &gt; <a href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a><br>
&gt; &gt;<br>
<br>
&gt; _______________________________________________<br>
&gt; sssd-devel mailing list<br>
&gt; <a href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a><br>
&gt; <a href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a><br>
<br>
_______________________________________________<br>
sssd-devel mailing list<br>
<a href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a><br>
</div></div></blockquote></div><br></div></div>