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

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...

Thanks.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Oct 3 22:48:15 2007