You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2011/01/13 12:07:27 UTC

svn commit: r1058503 - /subversion/site/publish/docs/release-notes/1.7.html

Author: stsp
Date: Thu Jan 13 11:07:27 2011
New Revision: 1058503

URL: http://svn.apache.org/viewvc?rev=1058503&view=rev
Log:
* publish/docs/release-notes/1.7.html: Remove false claim that the svnsync
   race can be avoided with 1.6 svnsync clients.

Modified:
    subversion/site/publish/docs/release-notes/1.7.html

Modified: subversion/site/publish/docs/release-notes/1.7.html
URL: http://svn.apache.org/viewvc/subversion/site/publish/docs/release-notes/1.7.html?rev=1058503&r1=1058502&r2=1058503&view=diff
==============================================================================
--- subversion/site/publish/docs/release-notes/1.7.html (original)
+++ subversion/site/publish/docs/release-notes/1.7.html Thu Jan 13 11:07:27 2011
@@ -647,11 +647,6 @@ This fixes a known race condition in the
 <code>svnsync</code> (see <a
 href="http://subversion.tigris.org/issues/show_bug.cgi?id=3546"
  >issue #3546</a>).</p>
-<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>
 
 </div>  <!-- atomic-revprops -->
 



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&nbsp;(dump)<br>1.7&nbsp;(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&nbsp;(dump)<br>1.7&nbsp;(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 Daniel Shahaf <d....@daniel.shahaf.name>.
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&nbsp;(dump)<br>1.7&nbsp;(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?