You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Sperling <st...@elego.de> on 2009/12/09 22:23:58 UTC

Re: svn commit: r888979 - in /subversion/trunk/subversion: libsvn_repos/load.c tests/cmdline/svnadmin_tests.py

On Wed, Dec 09, 2009 at 09:24:20PM -0000, cmpilato@apache.org wrote:
> Author: cmpilato
> Date: Wed Dec  9 21:24:19 2009
> New Revision: 888979
> 
> URL: http://svn.apache.org/viewvc?rev=888979&view=rev
> Log:
> Fix issue #3547 - 'svnadmin load --parent-dir PATH' can destroy mergeinfo

> +      /* The svn:mergeinfo property syntax demands absolute repository
> +         paths, so prepend a leading slash if   */

if... if... if... if what?
Did you mean to leave this comment incomplete?

But it's not too hard to guess what the missing part could be.
Quoting it verbatim would make for an entertaining warm-up question
during an "English for advanced learners" course. Is "svn:mergeinfo"
already listed in the oxford dictionary though?

Stefan

Re: svn commit: r888979 - in /subversion/trunk/subversion: libsvn_repos/load.c tests/cmdline/svnadmin_tests.py

Posted by "C. Michael Pilato" <cm...@collab.net>.
Stefan Sperling wrote:
> On Wed, Dec 09, 2009 at 09:24:20PM -0000, cmpilato@apache.org wrote:
>> Author: cmpilato
>> Date: Wed Dec  9 21:24:19 2009
>> New Revision: 888979
>>
>> URL: http://svn.apache.org/viewvc?rev=888979&view=rev
>> Log:
>> Fix issue #3547 - 'svnadmin load --parent-dir PATH' can destroy mergeinfo
> 
>> +      /* The svn:mergeinfo property syntax demands absolute repository
>> +         paths, so prepend a leading slash if   */
> 
> if... if... if... if what?
> Did you mean to leave this comment incomplete?

Doh!  r889081.

-- 
C. Michael Pilato <cm...@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand