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

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