Gentoo Archives: gentoo-catalyst

From: Matt Turner <mattst88@g.o>
To: gentoo-catalyst@l.g.o
Cc: Brian Dolbec <dolsen@g.o>
Subject: Re: [gentoo-catalyst] [PATCH 1/4] Whitespace cleanup.
Date: Thu, 28 Feb 2013 23:44:42
Message-Id: CAEdQ38ERFjc6UqAVQdNG4q5LhQQeqUjZqb7EXLhQfX5brzkyVw@mail.gmail.com
In Reply to: Re: [gentoo-catalyst] [PATCH 1/4] Whitespace cleanup. by "W. Trevor King"
1 On Thu, Feb 28, 2013 at 3:46 AM, W. Trevor King <wking@×××××××.us> wrote:
2 > On Wed, Feb 27, 2013 at 05:46:58PM -0800, Brian Dolbec wrote:
3 >> Run the following command to cleanup whitespace.
4 >> for FILE in $(git ls-tree -r --name-only HEAD | grep -v 'bz2$'); do
5 >> sed -i 's/[[:space:]]*$//' "$FILE"
6 >> done
7 >
8 > Since you dropped the changes to COPYING, you should probably add
9 > COPYING to the grep (grep -v 'bz2$\|COPYING') or add `git checkout
10 > COPYING` after the for loop.
11 >
12 >> Signed-off-by: Brian Dolbec <dolsen@g.o>
13 >
14 > I don't thing this is appropriate for catalyst, since we never
15 > document its meaning for this project. If we *do* want to add
16 > Linux/Git-style sign-offs, I've got the docs all pulled out into their
17 > own GPLv2 project [1], which could be merged into catalyst. In this
18 > case, I imagine it's not worth the trouble ;).
19
20 Yeah, I don't think S-o-B adds anything. I've stopped using them for
21 everything but the X server and kernel.