[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] configure: allow user to override cflags.
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] configure: allow user to override cflags. |
Date: |
Mon, 07 Feb 2011 16:08:50 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Anthony Liguori <address@hidden> writes:
> On 02/07/2011 08:05 AM, Tristan Gingold wrote:
>> In order to allow user to override cflags, predefined flags must be inserted
>> before user cflags.
>>
>> Signed-off-by: Tristan Gingold<address@hidden>
>>
>
> I think there's a very specific reason we do it this way but I cannot
> remember at the moment.
Commit 1e027be7e91d854d7a0132e4a32bdf222c33dcfe perhaps?
>
> Regards,
>
> Anthony Liguori
>
>> ---
>> configure | 6 ++++--
>> 1 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 598e8e1..f18ed0d 100755
>> --- a/configure
>> +++ b/configure
>> @@ -939,8 +939,10 @@ cat> $TMPC<< EOF
>> int main(void) { return 0; }
>> EOF
>> for flag in $gcc_flags; do
>> - if compile_prog "-Werror $QEMU_CFLAGS" "-Werror $flag" ; then
>> - QEMU_CFLAGS="$QEMU_CFLAGS $flag"
>> + if compile_prog "-Werror $flag $QEMU_CFLAGS" "" ; then
>> + # Note: flag must be prepended so that they could be overriden by
>> + # user flags (such as -fno-stack-protector)
>> + QEMU_CFLAGS="$flag $QEMU_CFLAGS"
>> fi
>> done
>>
>>