[PATCH] lib/util: Avoid splitting tevent-unix-util as public library

Jeremy Allison jra at samba.org
Fri Jun 3 18:21:30 UTC 2016


On Fri, Jun 03, 2016 at 02:21:59PM +1000, Amitay Isaacs wrote:
> On Thu, Jun 2, 2016 at 7:32 PM, Volker Lendecke <Volker.Lendecke at sernet.de>
> wrote:
> 
> > On Thu, Jun 02, 2016 at 05:49:54PM +1000, Amitay Isaacs wrote:
> > > Hi,
> > >
> > > Commit 670db6ac1d678babd25dd82c4467c0f094cfabc5 split tevent-util public
> > > library to create tevent-unix-util public library for standalone ctdb
> > > use.  This created a public library dependency between samba and ctdb
> > > for packaging.
> > >
> > > For now live with tevent-unix-util as a private library for ctdb.
> > > Bundle tevent_unix.c in public library tevent-util as before.
> > >
> > > Please review and push.
> >
> > Question -- to me this looks as if you can't link both. What happens
> > then for the tfd patches that might want to link against
> > tevent-unix-util and then are linked in by smbd that also links in
> > tevent-util. Haven't tried, but would that work?
> >
> 
> Good point.  I forgot about the pending tfd patches.
> 
> I have reworked my fixes and are attached as ctdb.patches.
> Now tevent-util is built as a private library for ctdb build with only
> tevent_unix.c.  This is the same trick we use for samba-util.
> 
> 
> I have also reworked your patches on top of the above fixes as vl.patches.
> These fixes have following changes:
> 
>  - drop one patch which changed dependency from tevent-util to
> tevent-unix-util
>  - rebase fix for my patch to expose few more subsystems
>  - I have added fixup patch as suggested by Jeremy (can be squashed in)

Oh - doh ! Sorry, I missed that comment :-).

Plese squash it in and ignore my previous email :-).



More information about the samba-technical mailing list