diff mbox

build: simplify Makefile.target around unnest-vars invocations

Message ID 1398790003-4962-1-git-send-email-pbonzini@redhat.com
State New
Headers show

Commit Message

Paolo Bonzini April 29, 2014, 4:46 p.m. UTC
No need to save/restore obj-y, we can just build all-obj-y incrementally.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 Makefile.target | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

Comments

Fam Zheng April 30, 2014, 1:46 a.m. UTC | #1
On Tue, 04/29 18:46, Paolo Bonzini wrote:
> No need to save/restore obj-y, we can just build all-obj-y incrementally.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Reviewed-by: Fam Zheng <famz@redhat.com>

> ---
>  Makefile.target | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/Makefile.target b/Makefile.target
> index 4de6374..67e391d 100644
> --- a/Makefile.target
> +++ b/Makefile.target
> @@ -141,10 +141,7 @@ endif # CONFIG_SOFTMMU
>  %/translate.o: QEMU_CFLAGS += $(TRANSLATE_OPT_CFLAGS)
>  
>  dummy := $(call unnest-vars,,obj-y)
> -
> -# we are making another call to unnest-vars with different vars, protect obj-y,
> -# it can be overriden in subdir Makefile.objs
> -obj-y-save := $(obj-y)
> +all-obj-y := $(obj-y)
>  
>  block-obj-y :=
>  common-obj-y :=
> @@ -154,11 +151,7 @@ dummy := $(call unnest-vars,.., \
>                 block-obj-m \
>                 common-obj-y \
>                 common-obj-m)
> -
> -# Now restore obj-y
> -obj-y := $(obj-y-save)
> -
> -all-obj-y = $(obj-y) $(common-obj-y)
> +all-obj-y += $(common-obj-y)
>  all-obj-$(CONFIG_SOFTMMU) += $(block-obj-y)
>  
>  ifndef CONFIG_HAIKU
> -- 
> 1.8.3.1
> 
>
diff mbox

Patch

diff --git a/Makefile.target b/Makefile.target
index 4de6374..67e391d 100644
--- a/Makefile.target
+++ b/Makefile.target
@@ -141,10 +141,7 @@  endif # CONFIG_SOFTMMU
 %/translate.o: QEMU_CFLAGS += $(TRANSLATE_OPT_CFLAGS)
 
 dummy := $(call unnest-vars,,obj-y)
-
-# we are making another call to unnest-vars with different vars, protect obj-y,
-# it can be overriden in subdir Makefile.objs
-obj-y-save := $(obj-y)
+all-obj-y := $(obj-y)
 
 block-obj-y :=
 common-obj-y :=
@@ -154,11 +151,7 @@  dummy := $(call unnest-vars,.., \
                block-obj-m \
                common-obj-y \
                common-obj-m)
-
-# Now restore obj-y
-obj-y := $(obj-y-save)
-
-all-obj-y = $(obj-y) $(common-obj-y)
+all-obj-y += $(common-obj-y)
 all-obj-$(CONFIG_SOFTMMU) += $(block-obj-y)
 
 ifndef CONFIG_HAIKU