[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


Received on 2010-06-28 21:52:29 CEST

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