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

Reid Spencer reid at x10sys.com
Thu Apr 21 13:36:31 PDT 2005

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


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>

More information about the llvm-dev mailing list