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

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

 



> -----Original Message-----
> From: Randy.Dunlap [mailto:[email protected]]
> Sent: Thursday, June 29, 2006 12:39 AM
> To: Matt LaPlante
> Cc: [email protected]
> Subject: Re: [Patch] Attack of "the the"s in /arch
> 
> On Thu, 29 Jun 2006 00:06:25 -0400 Matt LaPlante wrote:
> 
[me]
> 
> 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

I used git13 as well.  I've actually got some more typo fixes ready to
submit, so I will find a better mailing method separately, then post them.
Thanks for the feedback.

-
Matt


-
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