[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Enabling FSFS debug code in alpha releases (was: Re: 1.10.0-alpha1 is up for signing)

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Fri, 17 Feb 2017 06:43:41 +0000

Evgeny Kotkov wrote on Thu, Feb 16, 2017 at 23:55:19 +0300:
> Stefan Sperling <stsp_at_elego.de> writes:
>
> >> 1) enabling it on trunk permanently (and disabling it on stable branches)
> >>
> >> 2) tagging a wc that has the patch applied
> >>
> >> 3) committing the patch to a branch and <handwave>arrange for the tag to
> >> incorporate the changes from that branch</handwave>.
> >
> > 4) Encourage people to compile alpha/beta/rc with --enable-maintainer-mode.
>
> 4) seems like an appropriate option to me.
>
> This is because we only ship the source code, and I think that those who
> actually build it should be in charge of enabling or disabling the debug
> features.

So perhaps we should change the guard from
.
    #ifdef SVN_DEBUG
.
to
.
    #if defined(SVN_DEBUG) || defined(FSFS_VERIFY_AS_REVISION_BEFORE_CURRENT_PLUS_PLUS)
.
to make it easier to enable that one feature, without everything else
that maintainer mode brings.

Cheers,

Daniel
(and if we choose #4, we should document that somewhere — release notes
or release announcement or something)
Received on 2017-02-17 07:48:01 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.