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

Re: Strip trailing whitespace

From: Daniel Becroft <djcbecroft_at_gmail.com>
Date: Wed, 7 Oct 2009 09:14:47 +1000

On Tue, Oct 6, 2009 at 9:47 PM, Olaf van der Spek <olafvdspek_at_gmail.com>wrote:

> On Tue, Oct 6, 2009 at 12:56 PM, Simon Large
> <simon.tortoisesvn_at_googlemail.com> wrote:
> > That would have to be supported in the subversion library, like
> > svn:eol-style, and I don't think that will ever happen. It is not the
> > job of a version control system to change your files!
>
> Things like keywords also change files...
> A VCS is the central component that stores your files and as such not
> an unlogical place to do this kind of thing.
>

Keywords and svn:eol-style are conversion processes, not code-formatting
tools. Also, as far as I know, keyword expansion does not result in a
modified file state.

>
> The only reason
> > svn:eol-style is there is for multi-platform support.
>
> > You could add your own trailing space stripper as part of a pre-commit
> > or start-commit hook. The problem is that in changing the files it may
> > also change the modified status one way or the other.
>
> Aren't those hooks client-side as well? Disadvantage of that is
> ensuring every client has those hooks.
>
> Olaf
>
> ------------------------------------------------------
>
> http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2404031
>
> To unsubscribe from this discussion, e-mail: [
> dev-unsubscribe_at_tortoisesvn.tigris.org].
>

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2404306

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2009-10-07 04:54:50 CEST

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

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