[PATCH] lib/util: Build iov_buf library only when building samba

Michael Adam obnox at samba.org
Fri Feb 27 00:35:14 MST 2015


On 2015-02-27 at 16:38 +1100, Martin Schwenke wrote:
> On Thu, 26 Feb 2015 09:31:39 +0100, Michael Adam <obnox at samba.org>
> wrote:
> 
> > On 2015-02-26 at 08:30 +0100, Volker Lendecke wrote:
> > > On Thu, Feb 26, 2015 at 11:52:48AM +1100, Amitay Isaacs wrote:
> > > > Hi,
> > > > 
> > > > Adding new libraries in lib/util at the wrong place in wscript breaks
> > > > standalone ctdb rpm build.
> > > 
> > > Let's better revert this patch. Whenever I touch ANYTHING in
> > > lib/ I step on someones toes. See the unresolved debug
> > > desaster with OpenChange recently. I will send out reverts.
> > 
> > Don't be silly... I can't imagine on who's toes you
> > might have stepped. The only possible problem is that
> > the standalone ctdb build is apparently not tested as
> > well as it should be - not your fault.
> > 
> > I think Amitay's patch is perfectly reasonable.
> > Reviewed-by: /me
> > 
> > I might push later if there are no further objections.
> 
> I've pushed this.

Thanks!

> It is the only simple solution we have right now.

Agreed.

> Volker, please keep refactoring lib/util.  I love your work!  It is
> more important than occasional RPM build breakage in CTDB.

ACK!

> If someone happens to break it again then we'll just quietly push a similar
> fix to this one...
> 
> ... until we get time to handle this issue properly.  :-)

I think this summarizes and concludes it nicely, and we can
hope that Volker (and others) keep refactoring not only in
source3/ ... :-)

Cheers - Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150227/ab8700c3/attachment.pgp>


More information about the samba-technical mailing list