[PATCH] ctdb: ctdb_protocol.h includes "protocol/protocol.h"

[Martin Schwenke]

On Sat, 7 Nov 2015 23:30:32 +1100, Amitay Isaacs <amitay at gmail.com>
wrote:

> On Sat, Nov 7, 2015 at 10:13 AM, Martin Schwenke <martin at meltin.net> wrote:
> 
> > On Fri, 6 Nov 2015 14:54:39 +0100, Volker Lendecke
> > <Volker.Lendecke at SerNet.DE> wrote:
> >
> > > Review&push appreciated!
> >
> > I'm just wondering what is the best way of handling this...
> >
> > When --with-cluster-support/CLUSTER_SUPPORT is enabled then
> > ctdb/wscript is loaded and it does this, via the following recent
> > commit.
> >
> > commit 23932d38dd2c83460a34030dcbad2d70a5e0157d
> > Author: Amitay Isaacs <amitay at gmail.com>
> > Date:   Thu Oct 29 18:28:29 2015 +1100
> >
> >     ctdb-build: Add ctdb/ directory to include path for top-level build
> >
> >     Signed-off-by: Amitay Isaacs <amitay at gmail.com>
> >
> > Do we need to include any ctdb/... paths unconditionally in source3 or
> > should we just avoid including CTDB headers when CLUSTER_SUPPORT is not
> > set?
> >
> 
> There is no need to set CTDB specific flags and include in source3 build
> since we are not going to build against external CTDB.
> 
> How about the following patch?

I'm game...  looks sane to me... :-)

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

Pushed...  let's see if it lands!

peace & happiness,
martin