Subject: Re: [BUILD-FAILURE] linux-next: Tree for June 4 - firmware/keyspan/mpr.fw.gen.S

From: Adrian Bunk <[email-masked]>
Date: Wed, 4 Jun 2008 21:20:11 +0300

On Wed, Jun 04, 2008 at 06:52:10PM +0100, David Woodhouse wrote:
> On Wed, 2008-06-04 at 19:19 +0200, Segher Boessenkool wrote:
> > It is a new feature in GNU make 3.81, which was released slightly
> > over two years ago.
> 
> Hm, Ok. I'll stick in a workaround. Thanks.
> 
> diff --git a/firmware/Makefile b/firmware/Makefile
> index b4f758a..166a9b0 100644
> --- a/firmware/Makefile
> +++ b/firmware/Makefile
> @@ -83,6 +83,13 @@ wordsize_deps := $(wildcard include/config/64bit.h include/config/32bit.h \
>  		include/config/superh32.h include/config/superh64.h \
>  		include/config/x86_32.h include/config/x86_64.h)
>  
> +# Workaround for ancient versions of make, where .SECONDEXPANSION doesn't work.

Can you write make < 3.81 instead?

We do know the version and this will make it easier in a few years to 
determine when the workaround will no longer be required.

> +# It'll end up depending on these targets, so make them a PHONY rule which
> +# depends on _all_ the directories in $(firmware-dirs), and it'll work out OK.
> +PHONY += $(objtree)/$$(%) $(objtree)/$(obj)/$$(%)
> +$(objtree)/$$(%) $(objtree)/$(obj)/$$(%): $(firmware-dirs)
> +	@true
> +
>  # For the $$(dir %) trick, where we need % to be expanded first.
>  .SECONDEXPANSION:
>  
> dwmw2

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