[LLVMdev] Trailing whitespace removal (important for CVS users!)

[Reid Spencer]

Why not put all this into a pre-commit filter in CVS and be done with
it? We'd never be bothered with it again as it would never be committed
again.

Reid.

On Thu, 2005-04-21 at 15:11 -0500, Misha Brukman wrote:

> Dear LLVMers,
> 
> If you live on the bleeding edge (i.e. CVS version), please read!
> 
> On Wed, Apr 20, 2005 at 12:12:54PM +0200, Markus F.X.J. Oberhumer wrote:
> > Do you really want external patches for this ? A simple Perl script
> > that runs on all *.h and *.cpp files, and a local commit from your
> > side would be much simpler. 
> 
> I'm in the process of doing just this as we speak.  What this means:
> please hold off updating in the mean time, unless you want to rebuild
> your codebase several times, as I am about to touch a *lot* of files.
> 
> I will send a message to the list when I'm done, so you can get all the
> changes (and hopefully, few conflicts)
> 
> > The testsuite should also be enhanced to daily report files with
> > trailing whitespace. 
> 
> Good idea, I'll add that to my todo list.
> 
> > Please note that this not purely academic - trailing whitespace are a
> > horror for anyone maintaining external patches against a CVS tree.
> 
> I was not aware that this is such a big problem, so thanks for point
> this out.  It'll be fixed in a short while.
> 



_______________________
Reid Spencer
President & CTO
eXtensible Systems, Inc.
rspencer at x10sys.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20050421/610805ee/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20050421/610805ee/attachment.sig>