On Fri, 2009-01-09 at 09:41 -0800, John Gardiner Myers wrote:
> Julian Foad wrote:
> > I would want to check how it affects compatibility,
> >
> If an existing external merge tool checks the number of arguments to
> ensure there are exactly four, it would have to have this check removed
> or changed to ensure there are no fewer than four. I believe this is
> reasonable, as the alternative is to add an additional "merge2" config
> setting and environment variable to distinguish old and new merge tools.
>
> I plan in the follow-up modification to the Subversion Book to state
> that external merge tools need to ignore any additional arguments that
> may be added in future versions of Subversion.
Sounds good.
> > whether there are
> > other similar cases that should be handled too (interactive resolution
> > has a similar problem, for example,
> I don't understand the issue. I believe this change affects all
> invocations of the external merge tool, whether interactive or not.
> > and I think there's more than one
> > place where merge is called, I think) and so on.
> >
> I believe there are no issues with respect to merges from "svn merge"
> versus "svn update". All invocations should be modified by this patch.
To give a clue of the sort of things on my mind:
* There are separate "merge-cmd" and "diff3-cmd" config settings,
which both run a merge tool, perhaps in different ways, in different
situations. I think "diff3-cmd" is run before deciding whether there is
a conflict, and then "merge-cmd" is run if the user selects "Launch
external merge tool".
* The external diff tool has the same problem.
* The "diff" options in interactive conflict resolution print things
like "+++ tmp/tmpfile.tmp" in their headers. They could benefit from
knowing the real name of the text that it's diffing.
Regards,
- Julian
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1018700
Received on 2009-01-12 10:49:25 CET