[gPXE-devel] [Etherboot-developers] [PATCH] [makefile] Disable ccache for embedded.o

[Matt_Domsch at dell.com]

Thank you Stefan for tracking this down!  I thought I was seeing things after our IRC conversation about it. :-)

--
Matt Domsch
Technology Strategist, Dell Office of the CTO
linux.dell.com & www.dell.com/linux


-----Original Message-----
From: Marty Connor [mailto:mdc at etherboot.org] 
Sent: Tuesday, March 02, 2010 9:14 PM
To: Stefan Hajnoczi
Cc: Etherboot General Discussion List; Etherboot Developers List; gpxe-devel at etherboot.org
Subject: Re: [Etherboot-developers] [gPXE-devel] [PATCH] [makefile] Disable ccache for embedded.o

Wow.  Nice debugging, Stefan.   Seems like a nice improvement.

Looks like something we should definitely apply, when you're comfortable
with it.

Hmm, I also just noticed that hardly anyone has joined gpxe-devel.  I
think a message to etherboot-developers is in order.

Patches like this are too good to miss :)

Hey!  All you people on Etherboot-Developers, please join
gpxe-devel at etherboot.org (and gpxe at etherboot.org if you haven't)

>>>>  http://etherboot.org/mailman/listinfo <<<<

Don't miss out on quality conversation!  Join today! :)

/ Marty /

On 3/1/10 3:34 PM, Stefan Hajnoczi wrote:
> Embedded image support uses .incbin in inline assembly to include binary
> files.  The file dependency is not spotted by ccache when deciding
> whether or not to rebuild embedded.o.  This results in builds that
> contain an outdated version of the embedded image when ccache is used.
> 
> Reported-by: Tim 'Shaggy' Bielawa <tbielawa at jabber.org>
> Reported-by: Matt Domsch <Matt_Domsch at dell.com>
> Signed-off-by: Stefan Hajnoczi <stefanha at gmail.com>
> ---
>  src/Makefile.housekeeping |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/src/Makefile.housekeeping b/src/Makefile.housekeeping
> index 1f5e115..8ba7e44 100644
> --- a/src/Makefile.housekeeping
> +++ b/src/Makefile.housekeeping
> @@ -511,6 +511,13 @@ EMBED_ALL	:= $(foreach i,$(call seq,1,$(words $(EMBEDDED_FILES))),\
>  			     \"$(notdir $(word $(i),$(EMBEDDED_FILES)))\" ))
>  
>  $(BIN)/embedded.o : $(EMBEDDED_FILES) $(EMBEDDED_LIST)
> +
> +# This file uses .incbin inline assembly to include a binary file.
> +# Unfortunately ccache does not detect this dependency and caches builds even
> +# when the binary file has changed.
> +#
> +$(BIN)/embedded.o : override CC := env CCACHE_DISABLE=1 $(CC)
> +
>  CFLAGS_embedded = -DEMBED_ALL="$(EMBED_ALL)"
>  
>  # Generate the NIC file from the parsed source files.  The NIC file is


------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Etherboot-developers mailing list
Etherboot-developers at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/etherboot-developers