qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] virtio-9p: Fix compilation (wrong include state


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] virtio-9p: Fix compilation (wrong include statements)
Date: Thu, 28 Apr 2011 12:46:08 -0500
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.15) Gecko/20110419 Lightning/1.0b2 Thunderbird/3.1.9

On 04/28/2011 12:44 PM, Stefan Weil wrote:
Am 28.04.2011 19:39, schrieb Anthony Liguori:
On 04/28/2011 12:08 PM, Stefan Weil wrote:
From: Stefan Weil<address@hidden>

Commit 353ac78d495ef976242abd868f68d78420861c2c moved the files
without fixing the include paths.

Signed-off-by: Stefan Weil<address@hidden>


This is not correct. I think we learned in IRC that the problem is
Stefan is using CFLAGS=-g which overrides the CFLAGS append added by
this patch.

Regards,

Anthony Liguori


QEMU Makefiles use QEMU_CFLAGS - setting CFLAGS in Makefile* is wrong,
because now users can no longer set their own CFLAGS.

Use ./configure --extra-cflags="-g"

Regards,

Anthony Liguori


The special CFLAGS for virtio should be removed (with my patch applied
first).
The second best solution would be changing CFLAGS to QEMU_CFLAGS in
Makefile.objs and Makefile.target.

Regards,

Stefan






reply via email to

[Prev in Thread] Current Thread [Next in Thread]