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

Re: [PATCH]merge-tracking svn diff pretty output for svn:mergeinfo diffs

From: Madan U Sreenivasan <madan_at_collab.net>
Date: 2006-07-10 08:19:02 CEST

On Fri, 07 Jul 2006 19:35:17 +0530, Garrett Rooney
<rooneg@electricjellyfish.net> wrote:

> On 7/7/06, Madan U Sreenivasan <madan@collab.net> wrote:
>> On Fri, 07 Jul 2006 12:35:49 +0530, Kamesh Jayachandran
>> <kamesh@collab.net> wrote:
>>
>> > Hi All,
>> > As proposed by Garrett at
>> > http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=117410
>> > Find the attached patch.
>> > This patch makes the svn diff output to be human friendly to read
>> > something like the following,
>>
>> Did you see
>> http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=117467? I'm
>> -1 (FWIW ;) on this change.
>
> Another argument in favor of this change is that it actually does give
> you all the information, just in a more useful form for the user. I
> mean it's not like unified diff output includes all the data, just
> enough to make the diff apply, this particular kind of data doesn't
> actually need any context to apply the diff, so I don't see why we
> should be printing any.

Apologize for the delayed reply, was offline for the last two days.

Well, I have placed my points, and I beleive they have been received well.
Since I seem to be the only one objecting on this issue, (isn't this patch
getting committed because of my objection alone?), I take back my -1. Lets
say, I call it a -0 now! :) Thanks for the patch, Kamesh.

Regards,
Madan.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 10 07:48:54 2006

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