Re: [PATCH] kernel/module.c has something to hide. (whitespace cleanup)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 12 May 2005, Nick Piggin wrote:
> Bodo Eggert <[email protected]> wrote:
> >Jesper Juhl <[email protected]> wrote:

> >>If Andrew agrees, then I'll commit to doing this cleanup;

> >>- (to a limited degree) no trailing whitespace

> >I just ran a script over -rc4 to remove trailing ws. The result is
> >about 22 MB in 429 patches (iterated over ./*/*).
> >
> >How hard can I patch you before you start patching me?
> >
> >Which addresses am I supposed to send it to? I don't want to break the
> >record for the most annoying patch series in lkml.
> >
> 
> First of all, why is it 429 patches?

Because
1) some parts will get rejected due to conflicting patches. Only those 
parts will need to be recreated.
2) i forgot to create the 430th patch.

> The patches we want aren't about a
> file or a subdirectory or even a subsystem, but they're supposed to be
> a logical change. Ie. 1 patch.

That would be too large for most mailboxes. If you like a single patch,
you can just concatenate all the patches, so splitting it was a safe bet.

> An exception for something like this would
> be if you want to feed it to different maintainers seperately, but it
> sounds like you just want to bomb it somewhere...

I asume there is no automatic way to get the maintainer from a given 
file, and I don't want to grow old and grey while doing that manually.

-- 
"Bravery is being the only one who knows you're afraid."
-David Hackworth
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux