[PATCH] Remove ctdb feature checks from configure

Michael Adam obnox at samba.org
Fri Oct 24 04:33:18 MDT 2014


On 2014-10-24 at 11:49 +0200, Volker Lendecke wrote:
> On Fri, Oct 24, 2014 at 11:02:19AM +0200, Michael Adam wrote:
> > On 2014-10-24 at 10:51 +0200, Volker Lendecke wrote:
> > > On Fri, Oct 24, 2014 at 01:30:56PM +1100, Amitay Isaacs wrote:
> > > > This is a re-post since no one commented on these patches.
> > > > 
> > > > These patches remove the configure checks for ctdb features as we now have
> > > > the ctdb building with file server code.
> > > 
> > > Do we really want this? One checkin says that we always
> > > build with integrated ctdb. Our autobuild system requires
> > > that we still support the ctdb standalone as well as the
> > > samba build with external ctdb, so I think we can't drop
> > > these checks.
> > 
> > Well... that autobuild stuff was from the times before we had
> > a combined build. Implying that this requires keep Samba able
> > to build against external ctdb forever is not correct. After
> > all, autobuild is our script and it can be changed. And it
> > _should_ be changed if we go down the path Amitay proposes.
> > 
> > So you could turn your argument into stating that Amitay's patch
> > is simply not complete in that it forgot to change autobuild. :-)
> > 
> > All that being said, I am also a bit hesitant, but thinking
> > about it, it is really only consequent and consistent.
> 
> I've spent many hours in the last days trying to get parts
> of my messaging work through all the builds, and Martin has
> gone through great pains in there as well, and now you're
> telling me this might have all been bogus? Wow.

Am I? I don't know, since I didn't know about your build pains.

(And it is imho an extreme luxury to be able to complain
 about just a few hours spent in vain...)

The fact that autobuild is our own tool that we adapt to our
needs should not be new to either of you. ;-)

And after all, ctdb is still built, just as a aprt of
samba, and samba is still built against ctdb, so I can't
imagine which build problems you have been fighting with
that I have now talked into nonexistence. :-D

Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20141024/50f3a185/attachment.pgp>


More information about the samba-technical mailing list