fix - do not build blobs when blobs are not needed

Commit Message

Avi Kivity May 7, 2009, 9:26 a.m. UTC

Jes Sorensen wrote:
> Hi,
>
> Don't like ugly assembler errors because QEMU tries to build some x86
> assembly code which isn't needed on non-x86.

 .PHONY: kvm/extboot
 
+ifdef INSTALL_BLOBS
 all: kvm/extboot
 
 kvm/extboot:
@@ -427,3 +428,4 @@
            || ! cmp -s pc-bios/extboot.bin $@/extboot.bin; then \
 		cp $@/extboot.bin pc-bios/extboot.bin; \
 	fi
+endif


Seems a bit heavy handed.  blobs != extboot.

How about instead:

    build-targets-x86 = kvm/extboot

    all: $(build-targets-$(ARCH))

(or something similar that builds)

Patch

Index: qemu-kvm/configure
===================================================================
--- qemu-kvm.orig/configure
+++ qemu-kvm/configure
@@ -356,6 +356,7 @@ 
      cpu_emulation="no"
      gdbstub="no"
      slirp="no"
+     blobs="no"
 fi
 if [ "$cpu" = "powerpc" ]; then
      kvm="yes"