New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/27300#issuecomment-748714358

Comment:
I've done a little more digging into this and have made changes in my branch to address the following concerns:

1. The `ctdb` package should probably be called `samba-ctdb`. I wondered whether `ctdb` served a purpose as a standalone clustered database, but as upstream seems to think its sole purpose is to be tightly coupled with Samba, the name should probably reflect that it's a subcomponent.

2. Building with GlusterFS adds a hard dependency on `libglusterfs` for the main `samba` package. The only place this appears is in the `glusterfs.so` VFS module, so we can put this module and its man page into a separate subpackage that will allow people to avoid the dependency without requiring a custom build.

3. After review, I don't think `conf_files` or `mutable_files` are appropriate for contents of `/etc/ctdb`. The `functions` file seems to be sourced from a lot of script not intended to be edited by users (specifically, those in `/usr/share`) and probably should be considered non-editable. The `events` contents are symlinks, which don't work properly `conf_files`. I think the best option we have for these files is `noextract` rules if users want to override the default set of event scripts.

4. Some executables in `/usr/bin` related to CTDB were still left behind in the main `samba` package.

Please take a look at commit ahesford@d70038551bd0b5635ff893cc5e1b28afafd64d56 and, if you have no objections, I'll merge that one.