> From: Andrew Stitcher [mailto:[email protected]] > Sent: Monday, January 31, 2011 6:22 PM > To: [email protected] > Subject: RE: Zap trailing whitespace? > > > On Mon, 2011-01-31 at 18:17 -0500, Steve Huston wrote: > > > OK, let me try for consensus around this: > > > > > > Does anyone object to adding an svn hook that (1) strips trailing > > > whitespace, and (2) bans commits that contain tabs? > > > > I think I'd rather see the hook refuse to do the commit if > there are > > trailing whitespace or tabs. On the off chance that svn > gets it wrong > > for some odd file type, I'd rather final control be left in > a person's > > hands. > > I'm ok with this approach, but it still leaves the ultimate > control in the hook though! As it won't be possible to commit > anything it doesn't like, no matter how much you want!
Right, but I was thinking of the case where a new binary file, for example, had what appeared to be trailing whitespace. If the hook kicks it out, someone can fix the filter rather than try to figure out how the file got damaged later. -Steve --------------------------------------------------------------------- Apache Qpid - AMQP Messaging Implementation Project: http://qpid.apache.org Use/Interact: mailto:[email protected]
