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

Re: svn commit: r958671 - /subversion/trunk/subversion/libsvn_wc/adm_ops.c

From: Greg Stein <gstein_at_gmail.com>
Date: Mon, 28 Jun 2010 15:51:02 -0400

On Mon, Jun 28, 2010 at 15:24, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>...
>> > (on s.c.n viewvc's diffs used to ignore whitespace... doesn't seem to be
>> > the case on s.a.o though)
>>
>> Whitespace can be important (e.g Python scripts), so keeping simple
>> whitespace changes in the commit email is a Good Thing.
>
> +1.  (I'm used to being able to just pipe a commit mail to 'svn patch'.)
>
>> I hadn't realized that s.c.n did that.
>>
>
> In its default view, yes. (it also had a 'view raw unidiff' mode)

Whoops. I missed the "viewvc" part (rather than commit emails).
Yeah... whitespace elimination isn't bad there. Tho it could probably
examine the file, detect python code, and keep whitespace for those
files.

*shrug*

Thanks,
-g
Received on 2010-06-28 21:52:29 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.