[PATCHES] fix build with gpfs support

Martin Schwenke martin at meltin.net
Tue Jul 28 05:59:54 UTC 2015


On Tue, 28 Jul 2015 15:00:53 +1000, Martin Schwenke <martin at meltin.net>
wrote:

> On Tue, 28 Jul 2015 07:13:05 +0300, Alexander Bokovoy <ab at samba.org>
> wrote:
> 
> > On Mon, 27 Jul 2015, Bjoern Baumbach wrote:
> > > Hi!
> > > 
> > > The build with GPFS support is broken in 4.3.0.
> > > Particularly if the GPFS headers are not stored at "/usr/lpp/mmfs/include/".
> > > 
> > > I've attached a patch that allows to select the directory with
> > > a --with-gpfs=PATH option (patch 2).
> > > 
> > > Furthermore the GPFS headers are also needed to build the debug.c (patch 3).
> > Looks good to me. RB+, can we have another team review, please?
> 
> Reviewed-by: Martin Schwenke <martin at meltin.net>

Ironically, the CTDB standalone build failed in autobuild.

I think that in patch #2 the:

  opt.add_option('--with-gpfs',
  ...

hunk wants to go in lib/util/wscript, not source3/wscript.  Works for
me.

Please try again with that change...  Still...

Reviewed-by: Martin Schwenke <martin at meltin.net>

:-)

peace & happiness,
martin



More information about the samba-technical mailing list