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

Re: [PATCH] Pass the name of the conflicted file to the merge tool

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Tue, 13 Jan 2009 11:44:35 -0500

Julian Foad wrote:
> On Tue, 2009-01-13 at 10:09 -0500, C. Michael Pilato wrote:
>> Julian Foad wrote:
>>>> The existing code currently uses "-L" switches to pass labels into the
>>>> external diff3 and external diff tools. I don't think those labels have
>>>> relative path information, but I doubt we really want them to.
>>> I see C-Mike Pilato made an improvement yesterday to put make the
>>> temporary file names reflect the real file name, so hopefully that's
>>> made the situation better. I haven't looked at it yet.
>> I just tested this, and I think the change I made yesterday *does* help
>> this. I tested the (l)aunch option just now with Emacs as the "tool", and I
>> got a filename like <name-of-file>.tmp in Emacs. Is that the kind of
>> improvement folks were looking for?
>
> Mike, that's great.
>
> There may be an unintended change though: I noticed that the working
> file is now left as a zero-bytes file if the diff3 tool doesn't save a
> result file [1] and then you choose "postpone" at the prompt. :-(
>
> [1] (Maybe it's supposed to? mine's kdiff3 and doesn't if I quit it
> without saving.)

Er ... my change did nothing but tweak the tempfile path generation logic.
I don't see how that could have any negative side-effect unless we're
(wrongly) banking on filenames of a particular format elsewhere in the code.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1022253

Received on 2009-01-13 17:44:55 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.