[PATCH] snapper: add configure option to control build (default: auto)
David Disseldorp
ddiss at samba.org
Sun May 10 00:07:44 UTC 2020
Hi Matt,
On Fri, 8 May 2020 18:25:41 -0400, Matt Taylor via samba-technical wrote:
> As long as the option to disable is there that works for my scenario.
> I figured since it was automatically including snapper if dbus was
> found previously I would keep that automatic behavior.
>
> Updated patch attached. Snapper is enabled by default and configure
> will error out if dbus is not found and --disable-snapper was not
> passed.
There's some white-space damage below the new samba_add_onoff_option
line. The empty line between snapper and cephfs should also be removed.
Otherwise this looks fine to me:
Reviewed-by: David Disseldorp <ddiss at samba.org>
I expect the new default dbus dependency will catch a few people, but
the clear instructions on how to disable it should help.
As with your other patch, please use git format with a commit message
and sign off.
See https://wiki.samba.org/index.php/Contribute#Mailing_patches_to_samba-technical
> I'm contributing as an individual with no corporate
> involvement/restrictions, so does that mean no Signed-Off-by tag is
> needed? I have another trivial spelling patch to send after this.
As Andrew mentioned, the signed-off-by tag is still needed, but
otherwise this should be good to go.
Thanks for your contribution!
Cheers, David
More information about the samba-technical
mailing list