[PATCHES] CTDB - fix script and documentation paths at build/install time

[Michael Adam]

On 2015-09-10 at 20:18 +1000, Martin Schwenke wrote:
> On Tue, 8 Sep 2015 19:55:31 +1000, Martin Schwenke <martin at meltin.net>
> wrote:
> 
> > [...]
> 
> > Currently CTDB's scripts and documentation aren't consistent with
> > build/install-time settings for various paths.  This set of patches
> > attempts to fix that.  The first few patches are clean-ups and the
> > rest make sure everything is set nicely at build/install time. 
> > 
> > These patches are in my ctdb-script-paths branch:
> > 
> >   https://git.samba.org/?p=martins/samba.git;a=shortlog;h=refs/heads/ctdb-script-paths
> > 
> >   git://git.samba.org/martins/samba.git (ctdb-script-paths branch)
> > 
> > Review and push appreciated.  Thanks!
> 
> I've updated the branch above.  I won't clutter the list by reposting...
> 
> Changes from previous patchset are based on a partial review by Amitay.
> 
> Changes are:
> 
> * Tweak the wscript changes so variable names are simpler and are more
>   consistent with the way things are already done.
> 
> * Drop this patch:
> 
>     ctdb-scripts: Prefer configuration and initscripts in SYSCONFDIR
> 
>   My main concern here was to make sure the CTDB configuration can be
>   loaded regardless of installation prefix.  However, this is already
>   OK.  If the prefix is /usr/local then the configuration can live
>   in /usr/local/etc/ctdb/ctdbd.conf.  This works fine.
> 
>   This patch also tried to address ability to load configuration for
>   other subsystems (e.g. NFS) that aren't installed as part of a
>   distribution.  It didn't do a good job of that so I'm happy to make
>   another attempt at this later.
> 
>   The patch also had a bug.  :-)
> 
> Michael, I understand you'd like to review this.  If you're happy with
> it then please post a "Reviewed-by:" rather than pushing.  I'd also
> like to give Amitay a chance to comment on the last few patches that he
> didn't get to yesterday...  :-)

I will review and reply later..

Cheers - Michael

> I've tested this with the default prefix of /usr/local and starting
> CTDB with ctdbd_wrapper, as well as building RPMs using CTDB's
> packaging support.  I still need to test what happens in the case where
> xsltproc isn't available and perhaps make a minor change to the manpage
> handling.
> 
> Thanks!
> 
> peace & happiness,
> martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150910/77cb35cd/attachment.sig>