[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: Simon Large <simon.tortoisesvn_at_googlemail.com>
Date: Tue, 6 Oct 2009 16:18:48 +0100

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

> 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


:       ___
:  oo  // \\      "De Chelonian Mobile"
: (_,\/ \_/ \     TortoiseSVN
:   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
:   /_/   \_\     http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2009-10-06 17:18:56 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.