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

Re: [PATCH] Implement (l)aunch an external merge tool

From: Augie Fackler <durin42_at_gmail.com>
Date: 2007-10-04 00:14:27 CEST

On Oct 3, 2007, at 4:47 PM, Eric Gillespie wrote:

> Augie Fackler <durin42@gmail.com> writes:
>
>> I believe this addresses everything we talked about. The config
>> variable doesn't work yet, but I didn't fix it per our discussion on
>> IRC about you having added a fix for this in your wc.
>>
>> Attached the patch so that our mailserver can't munge it like it
>> evidently did last time.
>
> Looks good, except for some trailing whitespace, which I've
> cleaned up; that will give you conflicts after I commit, though.
> Running make check now, and suddenly wondering if some test needs
> to be updated to know about the default config file change...

I've actually thought about adding some tests to the interactive
conflict resolution using python and pexpect. Would it be alright to
add the dependency on pexpect in order to run those tests?
Thanks,
Augie

>
> Thanks.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Oct 4 00:14:40 2007

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