Hi Sumit,
Sorry for the delay, got sidetracked by trying to find an appropriate way
to handle the well known sids etc..
But thanks to your reminder I got over to do the integration into sssd's
autoconf/make now :)
So here is the patch as an attachment.
(as recommended by the contrib page - but if you like it better for review
l'll git send-email it as well..)
This patch does not include an update for the rpm specs.
Could you do that integration? For me as a gentoo user that is currently an
unknown-area :)
Regards, Ben
2013/10/9 Sumit Bose <sbose(a)redhat.com>
On Mon, Oct 07, 2013 at 12:22:13PM +0200, Benjamin Franzke wrote:
> 2013/10/7 Sumit Bose <sbose(a)redhat.com>
>
> > On Thu, Oct 03, 2013 at 11:57:07PM +0200, Benjamin Franzke wrote:
> > > >
> > > > Hi Ben,
> > > >
> > > > this is really great, thank you very much for the contribution! As
a
> > > > matter of fact, we've been planning on delivering the CIFS
plugin
as
> > > > part of the next release:
> > > >
https://fedorahosted.org/sssd/ticket/1534
> > > >
> > > > Have you found the design page we created?
> > > >
> >
https://fedorahosted.org/sssd/wiki/DesignDocs/IntegrateSSSDWithCIFSClient
> > >
> > >
> > > Yeah, I've read those pages before writing the plugin..
> > >
> > >
> > > >
> > > > Mostly the work on our side was done by Sumit Bose (I've CC-ed
him). I
> > > > would propose to get in touch with him to avoid duplicating some
> > effort.
> > > >
> > > Unfortunately Sumit is in Germany and there is a holiday so he won't
be
> > > > available until Monday.
> > > >
> > >
> > > > In general, I would suggest to merge this code into the SSSD tree
if
> > you
> > > > wouldn't mind. I think it would makes sense to have the code in
the
> > same
> > > > repo as the ID mapping library.
> > > >
> > >
> > > I'd really appreciate having it integrated in sssd tree,
> > > to have centralized maintainance and adoption by distributions..
> >
> > Thanks a lot for the patch. I would suggest to put your code into
> > src/lib/cifs_idmap_sss/ and modify SSSD's Makefile.am to build the
> > plugin. Would you like to do the integration and send the result for
> > review to this list again?
> >
>
> I'd like to do that integration, will send a patch..
I'd like to add the plugin to the next Fedora release which
unfortunately has its deadline next week. I don't want to rush you, but
do you have a chance to send the patch today or tomorrow? Otherwise
would you mind if I prepare the patch to integrate your code into the
SSSD tree?
Thanks for your help.
bye,
Sumit
>
>
> > About the well-know SIDs. They should not be handled inside the plugin
> > but by SSSD itself. If you are interested in adding them I'd be happy
to
> > help you how to start (if my help is needed at all :-).
> >
>
> Yup, sounds better to handle the SIDs by SSSD core. I'd like to try..
> I'll see whether time permits me to do that soon.. (doing this as a part
of
> my thesis)
>
>
>
> >
> > bye,
> > Sumit
> >
> > >
> > >
> > > > Thanks again for the contribution, much appreciated!
> > > > _______________________________________________
> > > > sssd-devel mailing list
> > > > sssd-devel(a)lists.fedorahosted.org
> > > >
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> > > >
> >
> > > _______________________________________________
> > > sssd-devel mailing list
> > > sssd-devel(a)lists.fedorahosted.org
> > >
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> >
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel(a)lists.fedorahosted.org
> >
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> >
> _______________________________________________
> sssd-devel mailing list
> sssd-devel(a)lists.fedorahosted.org
>
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel