Re: [Qemu-devel] qemu Makefile configure

[Bernhard Fischer <rep.nop@aon.at>]

On Wed, Jan 03, 2007 at 01:59:50PM +0000, Thiemo Seufer wrote:

>All of this works fine for me, but patches of this sort are likely to
>break things, so please test and comment.
>
>
>Thiemo
>

>Index: Makefile.target
>===================================================================
>RCS file: /sources/qemu/qemu/Makefile.target,v
>retrieving revision 1.134
>diff -u -p -r1.134 Makefile.target
>--- Makefile.target	7 Dec 2006 18:15:35 -0000	1.134
>+++ Makefile.target	3 Jan 2007 13:47:50 -0000
>@@ -91,66 +91,66 @@ endif
> endif
> 
> ifeq ($(ARCH),x86_64)
>-OP_CFLAGS=$(CFLAGS) -falign-functions=0
>-LDFLAGS+=-Wl,-T,$(SRC_PATH)/x86_64.ld
>+LDFLAGS+=-Wl,-T,$(SRC_PATH)/$(ARCH).ld
> endif
> 
> ifeq ($(ARCH),ppc)
-> CFLAGS+= -D__powerpc__
+ CPPFLAGS += -D__powerpc__

(doesn't cpp already define that on ppc?)

>Index: configure
>===================================================================
>RCS file: /sources/qemu/qemu/configure,v
>retrieving revision 1.115
>diff -u -p -r1.115 configure
>--- configure	1 Jan 2007 21:31:01 -0000	1.115
>+++ configure	3 Jan 2007 13:47:50 -0000
>@@ -243,6 +244,10 @@ for opt do
>   esac
> done
> 
>+# default flags for all hosts
>+CFLAGS=$CFLAGS "-Wall -O2 -g -fno-strict-aliasing"
>+LDFLAGS=$LDFLAGS "-g"

Does that really work for you?
I'd rather say
LDFLAGS="$LDFLAGS -g"

cheers,