diff mbox

[1/2] ppc64: Fix out-of-tree builds

Message ID 1304267036-28503-1-git-send-email-andreas.faerber@web.de
State New
Headers show

Commit Message

Andreas Färber May 1, 2011, 4:23 p.m. UTC
On ppc64 host, recursion into pc-bios/spapr-rtas/ fails for
out-of-tree builds. Add missing dir and symlink.

Cc: David Gibson <david@gibson.dropbear.id.au>
Cc: Alexander Graf <agraf@suse.de>
Signed-off-by: Andreas Färber <andreas.faerber@web.de>
---
 configure |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Andreas Färber May 1, 2011, 4:31 p.m. UTC | #1
Someone on Linux should check if a similar patch is necessary for roms/ 
SLOF/:

Am 01.05.2011 um 18:23 schrieb Andreas Färber:

> diff --git a/configure b/configure
> index 6f75e2e..491f77a 100755
> --- a/configure
> +++ b/configure
> @@ -3457,11 +3457,13 @@ done # for target in $targets
>
> # build tree in object directory in case the source is not in the  
> current directory
> DIRS="tests tests/cris slirp audio block net pc-bios/optionrom"
> +DIRS="pc-bios/spapr-rtas"
> DIRS="$DIRS roms/seabios roms/vgabios"

DIRS="$DIRS roms/SLOF"

And git-submodule shows me a fourth one there... roms/ipxe

> DIRS="$DIRS fsdev ui"
> FILES="Makefile tests/Makefile"
> FILES="$FILES tests/cris/Makefile tests/cris/.gdbinit"
> FILES="$FILES pc-bios/optionrom/Makefile pc-bios/keymaps"
> +FILES="$FILES pc-bios/spapr-rtas/Makefile"
> FILES="$FILES roms/seabios/Makefile roms/vgabios/Makefile"

FILES="$FILES roms/SLOF/Makefile"

> for bios_file in $source_path/pc-bios/*.bin $source_path/pc-bios/ 
> *.rom $source_path/pc-bios/*.dtb $source_path/pc-bios/openbios-*; do
>     FILES="$FILES pc-bios/`basename $bios_file`"
> -- 
> 1.7.3.4
David Gibson May 2, 2011, 1:09 a.m. UTC | #2
On Sun, May 01, 2011 at 06:23:55PM +0200, Andreas Färber wrote:
> On ppc64 host, recursion into pc-bios/spapr-rtas/ fails for
> out-of-tree builds. Add missing dir and symlink.

Ah, thank you.

> Cc: David Gibson <david@gibson.dropbear.id.au>
> Cc: Alexander Graf <agraf@suse.de>
> Signed-off-by: Andreas Färber <andreas.faerber@web.de>
> ---
>  configure |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/configure b/configure
> index 6f75e2e..491f77a 100755
> --- a/configure
> +++ b/configure
> @@ -3457,11 +3457,13 @@ done # for target in $targets
>  
>  # build tree in object directory in case the source is not in the current directory
>  DIRS="tests tests/cris slirp audio block net pc-bios/optionrom"
> +DIRS="pc-bios/spapr-rtas"

Surely it should be:
   DIRS="$DIRS pc-bios/spapr-rtas"
though.
Alexander Graf May 3, 2011, 10:18 a.m. UTC | #3
On 01.05.2011, at 18:31, Andreas Färber wrote:

> Someone on Linux should check if a similar patch is necessary for roms/SLOF/:

Mind to just send a patch I can try out with some instruction on how to trigger this? :)


Alex
Andreas Färber May 8, 2011, 4:07 p.m. UTC | #4
Am 03.05.2011 um 12:18 schrieb Alexander Graf:

> On 01.05.2011, at 18:31, Andreas Färber wrote:
>
>> Someone on Linux should check if a similar patch is necessary for  
>> roms/SLOF/:
>
> Mind to just send a patch I can try out with some instruction on how  
> to trigger this? :)

Well, using a build directory other than the source directory is  
common practice to either try multiple configurations (e.g., ppc and  
ppc64) or to be able to just `rm -rf' and start from scratch.

git clone git://git.qemu.org/qemu.git
mkdir qemu-something # or mkdir -p qemu/subdir
cd qemu-something
../qemu/configure ... && make
# observe error occurring without my patches
git am ...
# check if any further errors occur with my patches on Linux

During my attempts to get things compiling I did `git submodule init'.

$ git submodule
-d1d6b53b713a2b7c2c25685268fa932d28a4b4c0 roms/SLOF
-7aee315f61aaf1be6d2fff26339f28a1137231a5 roms/ipxe
-cc975646af69f279396d4d5e1379ac6af80ee637 roms/seabios
-19ea12c230ded95928ecaef0db47a82231c2e485 roms/vgabios

roms/{seabios,vgabios}/Makefile were being symlinked in configure, the  
other two not.

If I
$ git submodule update roms/SLOF
I see there is a roms/SLOF/Makefile, too. But at least with my two  
patches in place, SLOF is not being built automatically on Darwin.
David or Ben will be better suited for instructions on how to build  
SLOF. :)

Andreas
diff mbox

Patch

diff --git a/configure b/configure
index 6f75e2e..491f77a 100755
--- a/configure
+++ b/configure
@@ -3457,11 +3457,13 @@  done # for target in $targets
 
 # build tree in object directory in case the source is not in the current directory
 DIRS="tests tests/cris slirp audio block net pc-bios/optionrom"
+DIRS="pc-bios/spapr-rtas"
 DIRS="$DIRS roms/seabios roms/vgabios"
 DIRS="$DIRS fsdev ui"
 FILES="Makefile tests/Makefile"
 FILES="$FILES tests/cris/Makefile tests/cris/.gdbinit"
 FILES="$FILES pc-bios/optionrom/Makefile pc-bios/keymaps"
+FILES="$FILES pc-bios/spapr-rtas/Makefile"
 FILES="$FILES roms/seabios/Makefile roms/vgabios/Makefile"
 for bios_file in $source_path/pc-bios/*.bin $source_path/pc-bios/*.rom $source_path/pc-bios/*.dtb $source_path/pc-bios/openbios-*; do
     FILES="$FILES pc-bios/`basename $bios_file`"