[coreboot] [PATCH] flashrom: Clean up Makefile

Luc Verhaegen libv at skynet.be
Tue Jun 23 15:21:33 CEST 2009


On Tue, Jun 23, 2009 at 01:51:06PM +0200, Carl-Daniel Hailfinger wrote:
> On 23.06.2009 13:40, Luc Verhaegen wrote:
> > On Tue, Jun 23, 2009 at 12:49:55PM +0200, Carl-Daniel Hailfinger wrote:
> >   
> >> The makefile rules for %.o and flashrom.o are identical. Let %.o handle
> >> flashrom.o as well.
> >>
> >> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
> >>     
> > Acked-by: Luc Verhaegen <libv at skynet.be>
> >   
> 
> Thanks, committed in r626.
> 
> 
> > Why am i seeing your patches double in your emails? Np for small ones, 
> > but for a big patch, seeing it double might be confusing.
> >   
> 
> Some developers use Gmail and it seems Gmail mangles inline patches and
> doesn't display attached patches automatically. So the inline version if
> for reviewing and the attached version is for applying.
> If that problem doesn't exist anymore, I'll gladly skip attaching the
> patches and send them inline only.
> 
> Regards,
> Carl-Daniel

Oh, trusty old mutt here, with the setup i use, happily includes text 
attachments in the reply, and it nicely scrolls from the end of the 
actual mail straight into the attachment.

So I'd rather not see an inlined patch, and prefer just the attachment. 

You are aiding the people that are unable to deal with text attachments 
properly, but at the same time you are hurting those who do have that 
ability. Which of those two setups is "broken", and which of those are 
you spending some extra time on?

Luc Verhaegen.




More information about the coreboot mailing list