CTDB waf build

[Andrew Bartlett]

On Thu, 2014-06-19 at 15:59 +1000, Martin Schwenke wrote:
> On Wed, 18 Jun 2014 11:12:43 -0700, Amitay Isaacs <amitay at gmail.com>
> wrote:
> 
> > Here's the first draft of the migration from autoconf to waf build for CTDB.
> > 
> > Currently, CTDB is configured and built separately from Samba (top-level
> > build).  The integration of build requires some more work on lib/util, so
> > it can be used in CTDB without bringing in too many samba dependencies.
> > 
> > Since these changes drop duplicate libraries (talloc/tevent/tdb/replace), I
> > am not posting the patches on the mailing list.  The patches are available
> > in ctdb-build branch of my samba tree.
> > 
> > 
> > http://git.samba.org/?p=amitay/samba.git;a=shortlog;h=refs/heads/ctdb-build
> 
> Looks good to me.  The only nit is that:
> 
>     ctdb-mkversion: Support external VERSION specification
> 
> should go before:
> 
>     ctdb-build: Add waf build for CTDB
> 
> Reviewed-by: Martin Schwenke <martin at meltin.net>
> 
> However, I'm guessing that someone who knows waf should probably review
> too...  ;-)

It is pretty good, the main comment I would make is not to do this:

+    bld.SAMBA_SUBSYSTEM('ctdb-common',
+                        source=CTDB_COMMON_SRC,
+                        includes='include include/internal common .
lib/util',
+                        deps='replace popt talloc tevent tdb')
+

but to instead list the source files in source=, unless there is a
specific reason not to.  We have been moving away from this style
elsewhere in the code-base.

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba