<div dir="ltr">It only took me a year-and-a-half, but I committed a modified version of Daniel's patch today.  (Now it's "The Many Meanings of 'Lock'. :-))</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 17, 2016 at 4:09 PM, Daniel Shahaf <span dir="ltr"><<a href="mailto:danielsh@apache.org" target="_blank">danielsh@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Pavel Lyalyakin wrote on Thu, Mar 17, 2016 at 18:36:53 +0300:<br>
> Hello,<br>
><br>
> On Wed, Mar 9, 2016 at 4:24 PM, Daniel Shahaf <<a href="mailto:danielsh@apache.org">danielsh@apache.org</a>> wrote:<br>
><br>
> > Concerning the 'The Three Meanings of "Lock"' box:<br>
> > .<br>
> ><br>
> > <a href="http://svnbook.red-bean.com/nightly/en/svn.advanced.locking.html#svn.advanced.locking.meanings" rel="noreferrer" target="_blank">http://svnbook.red-bean.com/<wbr>nightly/en/svn.advanced.<wbr>locking.html#svn.advanced.<wbr>locking.meanings</a><br>
> > .<br>
> > Should that box mention svnsync locks as well?<br>
> ><br>
> > (These are the svn_ra__get_operational_lock() locks that svnsync takes<br>
> > using the svn:sync-lock revprop, exposed in the UI through the<br>
> > --disable-locks/--steal-lock options to 'svnsync sync'.  Note 'svnrdump<br>
> > load' also uses this kind of operational lock.)<br>
> ><br>
><br>
> I doubt that the revprop-based lock `svnsync` or `svnrdump` places is worth<br>
> mentioning in the 'The Three Meanings of "Lock"' section.<br>
><br>
> `svnsync` and `svnrdump` place `svn:sync-lock` and `svn:rdump-lock`<br>
> revprops to revision 0 and it's an implementation detail, not a real<br>
> concept.<br>
<br>
</span>That's a fair point, but on the other hand, this particular<br>
implementation detail is user-facing, via the --steal-lock option and<br>
the following error message:<br>
<br>
    % svnsync sync file://$PWD/{r,r2}<br>
    Failed to get lock on destination repos, currently held by 'hostname:84a4136e-a3fb-4f2b-<wbr>8b52-9502a886ff3a'<br>
    ⋮<br>
    Failed to get lock on destination repos, currently held by 'hostname:84a4136e-a3fb-4f2b-<wbr>8b52-9502a886ff3a'<br>
    svnsync: E000022: Couldn't get lock on destination repos after 10 attempts<br>
    zsh: exit 1     svnsync sync file://$PWD/{r,r2}<br>
<span class=""><br>
> It's just how it works in Subversion 1.9 and older versions.<br>
> Thinking about it, this could easily change in the future, should `svnsync`<br>
> or `svnrdump` stop placing such kind of locks.<br>
><br>
<br>
</span>If it changes in the future, we can update the book at that time to<br>
document the new behaviour...<br>
<br>
Thanks for the review,<br>
<br>
Daniel<br>
<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
svnbook-dev mailing list<br>
<a href="mailto:svnbook-dev@red-bean.com">svnbook-dev@red-bean.com</a><br>
<a href="http://www.red-bean.com/mailman/listinfo/svnbook-dev" rel="noreferrer" target="_blank">http://www.red-bean.com/<wbr>mailman/listinfo/svnbook-dev</a></div></div></blockquote></div><br></div>