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

Re: svn commit: r1398541 - /subversion/branches/1.7.x/STATUS

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Tue, 16 Oct 2012 02:41:57 +0200

philip_at_apache.org wrote on Mon, Oct 15, 2012 at 22:20:32 -0000:
> Author: philip
> Date: Mon Oct 15 22:20:32 2012
> New Revision: 1398541
>
> URL: http://svn.apache.org/viewvc?rev=1398541&view=rev
> Log:
> * STATUS: Clarify effect of r1237779.
>
> Modified:
> subversion/branches/1.7.x/STATUS
>
> Modified: subversion/branches/1.7.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/STATUS?rev=1398541&r1=1398540&r2=1398541&view=diff
> ==============================================================================
> --- subversion/branches/1.7.x/STATUS (original)
> +++ subversion/branches/1.7.x/STATUS Mon Oct 15 22:20:32 2012
> @@ -71,7 +71,7 @@ Candidate changes:
> +1: cmpilato
>
> * r1237779
> - Fix sense of property validation on load.
> + Fix sense of property validation on load and make the revprop change atomic.

The code uses either svn_repos_fs_change_rev_prop4() or
svn_fs_change_rev_prop2(). Both of them support atomic revprop changes,
but for both of them the code passes NULL to disable that functinoality.

IOW, that revision doesn't change atomicity, AFAICT.

> Justification:
> Validation is supposed to be on by default but is off, adding
> --bypass-prop-validation switches it on.
>
>
Received on 2012-10-16 02:42:35 CEST

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