You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2011/01/13 16:27:52 UTC
Re: svn commit: r1058503, 1058515
stsp@apache.org wrote on Thu, Jan 13, 2011 at 11:07:27 -0000:
> -<p>With a 1.7 server, it is possible to fix the svnsync race condition
> -even for pre-1.7 svnsync clients by installing the pre-revprop-change hook
> -script given in
> -<a href="http://subversion.tigris.org/issues/show_bug.cgi?id=3546#desc12"
> - >comment 12 of issue #3546</a>.</p>
>
This should be reverted, per Philip's recent comment.
> <td><a href="#svnrdump">svnrdump</a></td>
> <td>1.7</td>
> - <td>any</td>
> + <td>any (dump)<br>1.7 (load)</td>
> <td>any</td>
> <td><code>svnrdump load</code> should only be used with 1.7 servers
> to avoid a race condition (see <a href="#svnrdump">below</a>).</td></tr>
Why not state just "1.7" in the table, since Philip's hook should
prevent the race condition here too?
Re: svn commit: r1058503, 1058515
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
Got it done before the release: r1063420.
Stefan Sperling wrote on Thu, Jan 13, 2011 at 15:39:05 +0000:
> On Thu, Jan 13, 2011 at 05:27:52PM +0200, Daniel Shahaf wrote:
> > stsp@apache.org wrote on Thu, Jan 13, 2011 at 11:07:27 -0000:
> > > -<p>With a 1.7 server, it is possible to fix the svnsync race condition
> > > -even for pre-1.7 svnsync clients by installing the pre-revprop-change hook
> > > -script given in
> > > -<a href="http://subversion.tigris.org/issues/show_bug.cgi?id=3546#desc12"
> > > - >comment 12 of issue #3546</a>.</p>
> > >
> >
> > This should be reverted, per Philip's recent comment.
>
> Yes, I've seen it. Feel free to revert :)
>
> > > <td><a href="#svnrdump">svnrdump</a></td>
> > > <td>1.7</td>
> > > - <td>any</td>
> > > + <td>any (dump)<br>1.7 (load)</td>
> > > <td>any</td>
> > > <td><code>svnrdump load</code> should only be used with 1.7 servers
> > > to avoid a race condition (see <a href="#svnrdump">below</a>).</td></tr>
> >
> > Why not state just "1.7" in the table, since Philip's hook should
> > prevent the race condition here too?
>
> Fine with me.
Re: svn commit: r1058503, 1058515
Posted by Stefan Sperling <st...@elego.de>.
On Thu, Jan 13, 2011 at 05:27:52PM +0200, Daniel Shahaf wrote:
> stsp@apache.org wrote on Thu, Jan 13, 2011 at 11:07:27 -0000:
> > -<p>With a 1.7 server, it is possible to fix the svnsync race condition
> > -even for pre-1.7 svnsync clients by installing the pre-revprop-change hook
> > -script given in
> > -<a href="http://subversion.tigris.org/issues/show_bug.cgi?id=3546#desc12"
> > - >comment 12 of issue #3546</a>.</p>
> >
>
> This should be reverted, per Philip's recent comment.
Yes, I've seen it. Feel free to revert :)
> > <td><a href="#svnrdump">svnrdump</a></td>
> > <td>1.7</td>
> > - <td>any</td>
> > + <td>any (dump)<br>1.7 (load)</td>
> > <td>any</td>
> > <td><code>svnrdump load</code> should only be used with 1.7 servers
> > to avoid a race condition (see <a href="#svnrdump">below</a>).</td></tr>
>
> Why not state just "1.7" in the table, since Philip's hook should
> prevent the race condition here too?
Fine with me.