Paolo Bonzini <pbonz...@redhat.com> writes: > On 02/06/2015 10:01, Alex Bennée wrote: >> >> Michael Tokarev <m...@tls.msk.ru> writes: >> >>> 29.05.2015 19:29, Paolo Bonzini wrote: >>>> On 29/05/2015 16:14, Alex Bennée wrote: >>>>> You mean just do: >>>>> >>>>> diff --git a/configure b/configure >>>>> index b707429..f13831a 100755 >>>>> --- a/configure >>>>> +++ b/configure >>>>> @@ -353,7 +353,7 @@ for opt do >>>>> ;; >>>>> --cpu=*) cpu="$optarg" >>>>> ;; >>>>> - --extra-cflags=*) QEMU_CFLAGS="$optarg $QEMU_CFLAGS" >>>>> + --extra-cflags=*) QEMU_CFLAGS="$QEMU_CFLAGS $optarg" >>>>> EXTRA_CFLAGS="$optarg" >>>>> ;; >>>>> --extra-ldflags=*) LDFLAGS="$optarg $LDFLAGS" >>>>> >>>>> I guess at the time I was trying to be clean and avoiding multiple -O >>>>> calls. But I guess that will have the same effect. >>>> >>>> Yes, that. Most other QEMU_CFLAGS assignments add at the beginning, so >>>> I guess the remaining ones (including the --extra-cflags one) should too. >>> >>> So, what's the final version of this patch? >> >> I can send the second patch but I'm wary about changing all instances to >> QEMU_CFLAGS to append additional flags. > > Sure, that's a separate change.
Cool, sent to qemu-trvial: "[PATCH] configure: postfix --extra-cflags to QEMU_CFLAGS" > > Paolo -- Alex Bennée
