Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Lars Marowsky-Bree
On 2012-05-29T18:57:30, Florian Haas wrote: > > The integration with the cluster stack is rather specific to whatever > > pacemaker/corosync version + configuration you build against. > > Unfortunately. > Well that's what #ifdef HAVE_CLUSTER_STACK_H and friends are good for, no? I know. I just d

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Florian Haas
On Tue, May 29, 2012 at 6:38 PM, Lars Marowsky-Bree wrote: > On 2012-05-29T18:34:15, Florian Haas wrote: > >> Yeah, it seems you just broke the build by including cluster/stack.h >> and not bothering to add an AC_CHECK_HEADERS to configure.ac. Where >> does that come from, is that new to Pacemake

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Lars Marowsky-Bree
On 2012-05-29T18:34:15, Florian Haas wrote: > Yeah, it seems you just broke the build by including cluster/stack.h > and not bothering to add an AC_CHECK_HEADERS to configure.ac. Where > does that come from, is that new to Pacemaker? Uh? It builds here on the 1.1.7 pacemaker version. The integr

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Florian Haas
On Tue, May 29, 2012 at 6:26 PM, Lars Marowsky-Bree wrote: > On 2012-05-29T17:56:59, Florian Haas wrote: > >> In case you're wondering why I didn't use PKG_CHECK_MODULES for the PE >> libraries: their pkg-config file is currently broken; Andrew has a >> pull request for Pacemaker for that. > > I

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Lars Marowsky-Bree
On 2012-05-29T17:56:59, Florian Haas wrote: > In case you're wondering why I didn't use PKG_CHECK_MODULES for the PE > libraries: their pkg-config file is currently broken; Andrew has a > pull request for Pacemaker for that. I was wondering more about how to build this against older codebases, b

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Florian Haas
On Tue, May 29, 2012 at 4:32 PM, Lars Marowsky-Bree wrote: > On 2012-05-29T14:31:20, Florian Haas wrote: > >> Forgot to mention this in the original cover message, for those who >> haven't been following the discussion: this is for sbd which is just >> spinning off from cluster-glue. > > Thanks,

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Lars Marowsky-Bree
On 2012-05-29T14:31:20, Florian Haas wrote: > Forgot to mention this in the original cover message, for those who > haven't been following the discussion: this is for sbd which is just > spinning off from cluster-glue. Thanks, I've merged them both! Regards, Lars -- Architect Storage/HA

Re: [Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Florian Haas
On Tue, May 29, 2012 at 2:27 PM, Florian Haas wrote: > Lars, > > I did this as an exercise of sorts to understand how this compiles and > what its dependencies are. Considering the code base is quite small > it may seem like a pointless stunt to jump through all the autofoo > hoops, but it makes l

[Linux-ha-dev] [PATCH 1 of 2] build: autotoolize

2012-05-29 Thread Florian Haas
# HG changeset patch # User Florian Haas # Date 1338230941 -7200 # Branch autotools # Node ID 9888c2e4353b08599e6977e5e61dd6d34ce6151e # Parent c4de704b6cea21c69b3c767d1c47bed727f94d82 build: autotoolize diff -r c4de704b6cea -r 9888c2e4353b COPYING --- /dev/null Thu Jan 01 00:00:00 1970 +

[Linux-ha-dev] [PATCH 0 of 2] Autotoolize build

2012-05-29 Thread Florian Haas
Lars, I did this as an exercise of sorts to understand how this compiles and what its dependencies are. Considering the code base is quite small it may seem like a pointless stunt to jump through all the autofoo hoops, but it makes life that much easier for distro packagers. Feel free to apply th

Re: [Linux-ha-dev] [rfc] SBD with Pacemaker/Quorum integration

2012-05-29 Thread Lars Marowsky-Bree
On 2012-05-29T08:39:06, Florian Haas wrote: > > Should be packageable on every platform, though I admit that I've not > > tried building the pacemaker module against anything but the > > corosync+pacemaker+openais stuff we ship on SLE HA 11 so far. > Are you expecting this to build without "-I/us