qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: qemu-devel@nongnu.org
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	Avi Kivity <avi@redhat.com>, Alex Graf <agraf@suse.de>
Subject: [Qemu-devel] [PATCH 2/7] Always build roms in a separate directory
Date: Wed, 29 Jul 2009 13:45:54 -0500	[thread overview]
Message-ID: <1248893159-18785-2-git-send-email-aliguori@us.ibm.com> (raw)
In-Reply-To: <1248893159-18785-1-git-send-email-aliguori@us.ibm.com>

For some roms, the same source will get built multiple times for each board
type.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
---
 Makefile  |    9 ++++++---
 configure |   13 ++++++++++---
 2 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index c510ff3..059696a 100644
--- a/Makefile
+++ b/Makefile
@@ -12,7 +12,7 @@ config-host.mak:
 endif
 
 .PHONY: all clean cscope distclean dvi html info install install-doc \
-	recurse-all speed tar tarbin test
+	recurse-all speed tar tarbin test roms
 
 VPATH=$(SRC_PATH):$(SRC_PATH)/hw
 
@@ -220,9 +220,12 @@ clean:
 	rm -f slirp/*.o slirp/*.d audio/*.o audio/*.d block/*.o block/*.d
 	rm -f qemu-img-cmds.h
 	$(MAKE) -C tests clean
-	for d in $(TARGET_DIRS) $(ROMS) libhw32 libhw64; do \
+	for d in $(TARGET_DIRS) libhw32 libhw64; do \
 	$(MAKE) -C $$d $@ || exit 1 ; \
         done
+	for d in $(ROMS); do \
+	$(MAKE) -C pc-bios/$$d $@ || exit 1 ; \
+	done
 
 distclean: clean
 	rm -f config-host.mak config-host.h config-host.ld $(DOCS) qemu-options.texi qemu-img-cmds.texi
@@ -247,7 +250,7 @@ endif
 
 roms:
 	for d in $(ROMS); do \
-	$(MAKE) -C $$d || exit 1 ; \
+	$(MAKE) -C pc-bios/$$d || exit 1 ; \
         done
 
 install-doc: $(DOCS)
diff --git a/configure b/configure
index 264cc7b..f477aaf 100755
--- a/configure
+++ b/configure
@@ -1743,7 +1743,7 @@ echo "TOOLS=$tools" >> $config_host_mak
 roms=
 if test \( "$cpu" = "i386" -o "$cpu" = "x86_64" \) -a \
         "$targetos" != "Darwin" ; then
-  roms="roms/multiboot"
+  roms="multiboot"
 fi
 echo "ROMS=$roms" >> $config_host_mak
 
@@ -2109,13 +2109,20 @@ fi
 
 done # for target in $targets
 
+# make directories for roms
+for rom in $roms; do
+    mkdir -p pc-bios/$rom
+    rm -f pc-bios/$rom/Makefile
+    ln -s $source_path/roms/$rom/Makefile pc-bios/$rom/Makefile
+done
+
 # build tree in object directory if source path is different from current one
 if test "$source_path_used" = "yes" ; then
-    DIRS="tests tests/cris slirp audio block roms/multiboot"
+    DIRS="tests tests/cris slirp audio block"
     FILES="Makefile tests/Makefile"
     FILES="$FILES tests/cris/Makefile tests/cris/.gdbinit"
     FILES="$FILES tests/test-mmap.c"
-    FILES="$FILES roms/multiboot/Makefile pc-bios/keymaps pc-bios/video.x"
+    FILES="$FILES pc-bios/keymaps pc-bios/video.x"
     for bios_file in $source_path/pc-bios/*.bin $source_path/pc-bios/*.dtb $source_path/pc-bios/openbios-*; do
         FILES="$FILES pc-bios/`basename $bios_file`"
     done
-- 
1.6.2.5

  reply	other threads:[~2009-07-29 18:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-29 18:45 [Qemu-devel] [PATCH 1/7] Rename pc-bios/optionrom -> roms/multiboot Anthony Liguori
2009-07-29 18:45 ` Anthony Liguori [this message]
2009-07-29 18:45 ` [Qemu-devel] [PATCH 3/7] If we built a rom, install it. Otherwise, use shipped copy Anthony Liguori
     [not found]   ` <m37hxr46rt.fsf@neno.mitica>
2009-07-29 22:16     ` [Qemu-devel] " Anthony Liguori
2009-07-30  9:34       ` Avi Kivity
2009-07-30  9:39         ` Alexander Graf
2009-07-30 10:07           ` Avi Kivity
2009-07-30 10:04             ` Alexander Graf
2009-07-29 18:45 ` [Qemu-devel] [PATCH 4/7] Allow a cross compiler to be used for rom builds Anthony Liguori
2009-07-29 18:45 ` [Qemu-devel] [PATCH 5/7] Add proper headers to all BIOS patches Anthony Liguori
2009-07-29 18:45 ` [Qemu-devel] [PATCH 6/7] Add pc-bios as a git submodule Anthony Liguori
2009-07-29 21:25   ` [Qemu-devel] " Alexander Graf
2009-07-29 21:40     ` Anthony Liguori
2009-07-29 21:51       ` Alexander Graf
2009-07-29 22:10   ` [Qemu-devel] " Jordan Justen
2009-07-29 22:13     ` Anthony Liguori
2009-07-29 18:45 ` [Qemu-devel] [PATCH 7/7] Remove BIOS patches from the queue and update the README Anthony Liguori
2009-07-30  9:38   ` [Qemu-devel] " Avi Kivity
     [not found] ` <m3eirz474o.fsf@neno.mitica>
2009-07-29 19:35   ` [Qemu-devel] Re: [PATCH 1/7] Rename pc-bios/optionrom -> roms/multiboot Anthony Liguori
2009-07-30  9:38   ` Avi Kivity

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1248893159-18785-2-git-send-email-aliguori@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=agraf@suse.de \
    --cc=avi@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).