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