[PATCHES] CTDB - fix script and documentation paths at build/install time
Martin Schwenke
martin at meltin.net
Thu Sep 10 10:18:58 UTC 2015
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'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
More information about the samba-technical
mailing list