Re: [PATCH/RFC] doc: about email clients for Linux kernel patches

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

 



On Wed, Sep 12, 2007 at 09:53:00PM +0200, Peter Zijlstra wrote:
> 
> On Wed, 2007-09-12 at 15:16 -0400, Rik van Riel wrote:
> > Jeff Garzik wrote:
> > > Chris Friesen wrote:
> > >> Randy Dunlap wrote:
> > >>
> > >>> +Thunderbird (GUI)
> > >>> +
> > >>> +By default, thunderbird likes to mangle text, but there are ways to
> > >>> +coerce it into being nice.
> > >>
> > >> Can someone describe the problems with just attaching the patch in 
> > >> Thunderbird?  It's what Martin says he does on the linked document...
> > > 
> > > Email clients don't like to quote attachments, even text/plain ones, 
> > > which then makes attached patches much more difficult to review and 
> > > comment on (i.e. you greatly reduce the number of reviewers).
> > 
> > Interestingly, Thunderbird does this right and simply
> > adds text/plain attachments to the quoted text.
> 
> Devolution allows the same, but most other mailers dont. Esp the text
> based onces which are the majority under the people you want reviews
> from.

I still prefer patches directly inline, but mutt both is a text based 
MUA and defaults to adding text/plain attachments to the quoted text.

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
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