Re: svn_client_log3() doesn't work as advertised

Philip Martin wrote:
> Stefan Küng <tortoisesvn@gmail.com> writes:
>
>> svnadmin create repo
>> svn co file:///d:/repo wc
>> cd wc
>> svn mkdir trunk
>> echo test > trunk\file
>> svn add trunk\file
>> svn ci -m "log 1"
>> echo test2 > trunk\file
>> svn ci -m "log 2"
>> svn ren trunk\file trunk\file2
>> svn ci -m "log 3"
>> echo test > trunk\file2
>> svn ci -m "log 4"
>>
>> now an
>> svn_client_log3("d:/wc/trunk/file2", 4, 2, 1, 100, true, false,
>> callback, (void*)this, &ctx, pool);
>> will fail with
>> "unable to find repository location for file:///d:/repo/trunk/file2 in
>> revision 2".
>>
>> So the peg revision is not used at all here.
>>
>> Can someone please have a look at this? Better now than after
>> releasing 1.4.0.
>
> Using trunk on Linux:
>
> $ svn log wc/trunk/file2@4 -r2:1 --limit 100 --verbose
> ------------------------------------------------------------------------
> r2 | pm | 2006-04-19 18:58:01 +0100 (Wed, 19 Apr 2006) | 1 line
> Changed paths:
> M /trunk/file
>
> log 2
> ------------------------------------------------------------------------
> r1 | pm | 2006-04-19 18:57:45 +0100 (Wed, 19 Apr 2006) | 1 line
> Changed paths:
> A /trunk
> A /trunk/file
>
> log 1
> ------------------------------------------------------------------------

The same happens with the SVN client on windows.
But I suspect that it simply ignores the peg revision you pass on the
command line - you'll get the same if you don't pass it.

The problem really is in the svn_client_log3() function.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org