[PATCH] snapper: add configure option to control build (default: auto)

Andrew Bartlett abartlet at samba.org
Fri May 8 22:35:54 UTC 2020


Thanks.  We just need that declaration and we won't be looking for the
DCO paperwork, but please do still include the Signed-off-by.

Thanks!

Andrew Bartlett

On Fri, 2020-05-08 at 18:25 -0400, Matt Taylor 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.
> 
> 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.
> 
> Thanks,
> 
> -Matt
> 
> On Fri, May 8, 2020 at 4:36 PM Andrew Bartlett <abartlet at samba.org>
> wrote:
> > 
> > On Fri, 2020-05-08 at 22:24 +0200, David Disseldorp via samba-
> > technical
> > wrote:
> > > On Fri, 8 May 2020 14:05:16 -0400, Matt Taylor via samba-
> > > technical
> > > wrote:
> > > 
> > > > 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:
> > > https://www.samba.org/samba/devel/copyright-policy.html
> > 
> > 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
> > unintentionally lost.
> > 
> > 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
> > flag.
> > 
> > Andrew Bartlett
> > 
> > --
> > Andrew Bartlett                       https://samba.org/~abartlet/
> > Authentication Developer, Samba Team  https://samba.org
> > Samba Developer, Catalyst IT
> > https://catalyst.net.nz/services/samba
> > 
> > 
> > 
-- 
Andrew Bartlett                       https://samba.org/~abartlet/
Authentication Developer, Samba Team  https://samba.org
Samba Developer, Catalyst IT          
https://catalyst.net.nz/services/samba






More information about the samba-technical mailing list