[coreboot] [PATCH] flashrom: Clean up Makefile
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.
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?
More information about the coreboot