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 [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]
