[PATCH] snapper: add configure option to control build (default: auto)
abartlet at samba.org
Fri May 8 20:36:44 UTC 2020
On Fri, 2020-05-08 at 22:24 +0200, David Disseldorp via samba-technical
> On Fri, 8 May 2020 14:05:16 -0400, Matt Taylor via samba-technical
> > Recently the samba packages on Gentoo Linux had a hard dependency
> > on
> > dbus added. This is because the vfs_snapper module is enabled
> > automagically on the existence of dbus headers with no option to
> > disable it. See: https://bugs.gentoo.org/721320 for the Gentoo bug
> > where it was suggested this be reported upstream.
> > This patch adds a configure option to control the build of snapper
> > similar to other vfs modules.
> > Signed-off-by: Matt Taylor <liverbugg at rinux.org>
> Thanks for the patch Matt!
> The change looks reasonable, although I think I'd prefer to see the
> default be True, so that existing packagers aren't affected.
> Also, if this work is contributed on behalf of your employer, you'll
> need to sign the DCO as described at:
I agree. I do not like ./configure parameters that default to 'auto'
(and the same goes for features), because of the inverse of this, that
the absense of a header on a build machine would mean that a feature is
The recent practice has been to have a configure option to allow a
distribution to disable a feature (as is desired here) but to otherwise
give an error if the header/library is not available and point to the
Andrew Bartlett https://samba.org/~abartlet/
Authentication Developer, Samba Team https://samba.org
Samba Developer, Catalyst IT
More information about the samba-technical