Re: [Patch] Attack of "the the"s in /arch

["Randy.Dunlap" <rdunlap@xenotime.net>]

On Thu, 29 Jun 2006 00:06:25 -0400 Matt LaPlante wrote:

> Hi All,
>   This is my first attempt at submitting a patch, so I apologize if I've
> broken any rules.  I started out looking for errors in KConfig text, and
> after finding one instance of the popular repetition "the the" I decided to
> grep for other occurrences.  To my surprise, I popped up quite a few of
> them!  The patch below corrects these typos across several files, both in
> source comments and KConfig files.  There is no actual code changed, only
> text.  Note this only affects the /arch directory, and I believe I could
> find many more elsewhere. :) I'm not submitting for inclusion yet, so I can
> get feedback incase I need to change my format.

Hi Matt,

The only problem that I see is that your mail client or server
breaks some (longer) lines where they should not be split,
so the patch cannot be applied with 'patch'.

E.g.:

> diff -ru a/arch/arm/mach-lh7a40x/arch-lpd7a40x.c
> b/arch/arm/mach-lh7a40x/arch-lpd7a40x.c
> --- a/arch/arm/mach-lh7a40x/arch-lpd7a40x.c	2006-06-28
> 23:20:26.000000000 -0400
> +++ b/arch/arm/mach-lh7a40x/arch-lpd7a40x.c	2006-06-28
> 23:45:49.000000000 -0400

Above should be 3 lines:
diff ...
--- ...
+++ ...

but it is broken into 6 lines.  Your options are (e.g.):

- use (some) Linux client to send email (not all are good for patches)
- on Windows:  use thunderbird or sylpheed or (last resort, not
  really good since it makes reviewing & sending feedback on patches
  more difficult on us) is to use attachments.  Please don't go for
  the last resort.

Sylpheed (sylpheed.good-day.net) supports inserting a file into
the email body, which is perfect (at least on Linux it does,
I hope that it does on Windows too).
Tbird is more of a problem, but can do done.  If you care to use
tbird, take a look at
http://mbligh.org/linuxdocs/Email/Clients/Thunderbird
and
http://lkml.org/lkml/2005/12/27/191
and
http://lists.osdl.org/pipermail/kernel-janitors/2006-June/006478.html
for details on how to use it (on Linux, hopefully Windows is
about the same).

Try not to use attachments...

Oh, there are other split lines in your patch.  After joining all of
the split lines (10-12 of them), the patch applies cleanly except
for one file.  I'm applying it to 2.6.17-git13.  What kernel version
did you use to make it?  You should use a very current version.

---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/