Re: Strip trailing whitespace

2009/10/6 Olaf van der Spek <olafvdspek_at_gmail.com>:
> 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...

That is a subversion feature. TortoiseSVN does not touch keywords. As
I said before, if you want subversion to change your files for you, go
ask on the subversion users list ... and be prepared for the same
response.

> A VCS is the central component that stores your files and as such not
> an unlogical place to do this kind of thing.

I store some of my files centrally on a file server. I don't expect it
to sort out the problems with my editor. Why should a version control
system take it upon itself to correct those problems?

Read the subversion book on the subject of server side hooks. They
specifically advise against modifiying files in a pre-commit hook,
although it is not prohibited. Th problem is, apart from comitting
something different from what the developer actually saved, is that
the copy on the server and the local copy are now different, even
though they are at the same revision. This makes it difficult for
subversion to synchronise local and remote copies.

If you do as Jean-Marc says and reject commits with trailing spaces,
you programmers will soon set their editors up the way you want them
to.

Simon

-- 
:       ___
:  oo  // \\      "De Chelonian Mobile"
: (_,\/ \_/ \     TortoiseSVN
:   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
:   /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2404141
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].