fix - do not build blobs when blobs are not needed

Commit Message

Jes Sorensen May 7, 2009, 9:35 a.m. UTC

Avi Kivity wrote:
> Seems a bit heavy handed.  blobs != extboot.
> 
> How about instead:
> 
>    build-targets-x86 = kvm/extboot
> 
>    all: $(build-targets-$(ARCH))
> 
> (or something similar that builds)

There's no problem that cannot be fixed by applying an appropriately
large hammer .... :-)

This one works for me.

Jes

Comments

Avi Kivity May 7, 2009, 9:49 a.m. UTC | #1

Jes Sorensen wrote:
> There's no problem that cannot be fixed by applying an appropriately
> large hammer .... :-)
>
> This one works for me.

applied, thanks.

Patch

Do not try to build extboot on non-x86.

Signed-off-by: Jes Sorensen <[e-mail]>

---
 Makefile |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Index: qemu-kvm/Makefile
===================================================================
--- qemu-kvm.orig/Makefile
+++ qemu-kvm/Makefile
@@ -419,7 +419,10 @@ 
 
 .PHONY: kvm/extboot
 
-all: kvm/extboot
+build-targets-x86  = kvm/extboot
+build-targets-ia64 =
+
+all: $(build-targets-$(ARCH))
 
 kvm/extboot:
 	$(MAKE) -C $@