From: Dave A. <ai...@li...> - 2004-09-08 01:34:31
|
> checking anything in. I'd prefer that route since it enforces style > rules. There's an approved set of parameters for using ident on the > kernel somewhere. > scripts/Lindent, I used it on the i915.. Dave. > > On Wed, 8 Sep 2004 00:47:24 +0100 (IST), Dave Airlie <ai...@li...> wrote: > > > > > I used sed: > > > sed -e 's/[ ]*$//' < $TMP > $1 > > > > > > to strip the trailing white space from all of the DRM files. > > > Any objections to a check in? Kernel rules for patches are no trailing > > > white space. > > > > Hold off for a while, the problem with this is generating diffs against > > the kernel gets messy, if we do it I think it needs to happen to both > > trees at once, there are a lot of whitespace issues in the DRM, one > > superpatch is waay more acceptable from a version control point of view.. > > (Larry says you should never do whitespace fixes using BK unless you > > happen to be working on the code in that area... as you lose blame > > annotations for pieces of code).. > > > > Dave. > > > > -- > > David Airlie, Software Engineer > > http://www.skynet.ie/~airlied / airlied at skynet.ie > > pam_smb / Linux DECstation / Linux VAX / ILUG person > > > > > > > > -- David Airlie, Software Engineer http://www.skynet.ie/~airlied / airlied at skynet.ie pam_smb / Linux DECstation / Linux VAX / ILUG person |