[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn_client_diff4 and valid relative path throws error even though it shouldn't

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-11-23 21:14:29 CET

Mark Phippard wrote:
> On Nov 23, 2007 2:54 PM, Stefan Küng <tortoisesvn@gmail.com> wrote:
>> Just updated to r27993 of the svn trunk. The new param relative_to_dir
>> in svn_client_diff4() is nice, but it doesn't work in some situations:
>>
>> a diff between a working copy file and it's base throws an error like this:
>> Path 'C:\Users\kueng\AppData\Local\Temp/svndiff.tmp' must be an
>> immediate child of the directory
>> 'D:/Development/SVN/TortoiseSVN/ext/Subversion'
>>
>> The problem: I didn't specify any temp file path. That path comes from
>> the fact that the diff first 'translates' the base file (line endings,
>> maybe keywords, ...), and by doing so creates that temporary file.
>>
>> I would expect that since it was the svn function itself that created
>> that temp file, it shouldn't error out but check the original path instead.
>>
>> To reproduce:
>> * modify the file subversion\libsvn_subr\config_win.c
>> * call svn_client_diff4(path, REVWC, path, REVBASE,
>> 'subversion\libsvn_subr', ...)
>
> Stefan,
>
> Take a look at the latest patch in Issue#: 2723
>
> That seems to fix it for me.

Bingo! That fixes the problem.

I wonder though: why do full committers attach a patch to an issue
instead of just committing it? When I discovered the problem, the first
thing I did was updating to the latest HEAD revision to see if it might
have been fixed already. Even though I knew about the issue with the
'absolute paths in diff' because I saw the mails about this, it never
occurred to me to check the issue for a patch - I assumed the issue was
fixed when the new API was introduced.

At least now I know that I have to check the issues for patches first ;)

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Nov 23 21:14:45 2007

This is an archived mail posted to the Subversion Dev mailing list.