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

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

From: Ben Reser <ben_at_reser.org>
Date: Mon, 24 Dec 2012 08:20:41 -0800

On Sat, Dec 22, 2012 at 3:38 PM, <rhuijben_at_apache.org> wrote:
> Author: rhuijben
> Date: Sat Dec 22 23:38:08 2012
> New Revision: 1425371
>
> URL: http://svn.apache.org/viewvc?rev=1425371&view=rev
> Log:
> * STATUS: Update r1425368 justification
>
> 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=1425371&r1=1425370&r2=1425371&view=diff
> ==============================================================================
> --- subversion/branches/1.7.x/STATUS (original)
> +++ subversion/branches/1.7.x/STATUS Sat Dec 22 23:38:08 2012
> @@ -94,7 +94,8 @@ Candidate changes:
> * r1425368
> Use the right api for generating the repository dirent in mod_dav_svn.
> Justification:
> - Using the wrong api might cause assertions. Especially in maintainer mode.
> + Using dav_svn_get_repos_path() causes an assertion on Windows. This
> + public api was unused until the in-repo-authz was merged to trunk.

That's not actually true. It was used in mod_authz_svn, but only when
you provided a AuthzSVNReposRelativeAccessFile directive.

I'd bet if you tested on Windows you'd find that you can reproduce the
assertion with AuthzSVNReposRelativeAccessFile on 1.7.x.
Received on 2012-12-24 17:21:22 CET

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