From: Stefan Weil <weil@mail.berlios.de>
To: malc <av1474@comtv.ru>
Cc: QEMU Developers <qemu-devel@nongnu.org>, herbszt@gmx.de
Subject: [Qemu-devel] Re: [PATCH] Fix build for mingw32 on windows ($@ in macro)
Date: Fri, 20 Nov 2009 07:40:39 +0100 [thread overview]
Message-ID: <4B0639E7.40509@mail.berlios.de> (raw)
In-Reply-To: <Pine.LNX.4.64.0911200030570.2699@linmac.oyster.ru>
malc schrieb:
> On Thu, 19 Nov 2009, Stefan Weil wrote:
>
>> Make using mingw32 on windows does not preserve $@ in macros
>> when they are modified using this pattern:
>> target: macro += something
>>
>> This behaviour results in an error when QEMU_CFLAGS containing
>> "-MMD -MP -MT $@" is modified for compilation of source files
>> which use SDL: $@ will expand to nothing, -MT no longer has
>> the correct argument (it will take the next one from the command
>> line) and the build will fail or run with a wrong command line.
>>
>> The problem is fixed by using a new macro QEMU_DGFLAGS
>> which is not modified by a target rule.
>
> Why not just stuff `-MMD -MP -MT $@' into the rules?
>
> [..snip..]
As in most cases, there are many ways to do something...
During my test, I had the dependency flags in the rules.
I decided to use a macro to allow people running make
without dependency generation (make QEMU_DGFLAGS=).
Stefan
next prev parent reply other threads:[~2009-11-20 6:40 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-14 16:39 [Qemu-devel] Build failure on mingw Sebastian Herbszt
[not found] ` <Pine.LNX.4.64.0911142108210.2712@linmac.oyster.ru>
2009-11-16 19:38 ` [Qemu-devel] " Sebastian Herbszt
2009-11-16 19:54 ` malc
2009-11-17 21:22 ` Sebastian Herbszt
2009-11-19 19:07 ` [Qemu-devel] [PATCH] Fix build for mingw32 on windows ($@ in macro) Stefan Weil
2009-11-19 20:13 ` [Qemu-devel] " Sebastian Herbszt
2009-11-19 21:32 ` malc
2009-11-20 6:40 ` Stefan Weil [this message]
2009-11-20 1:26 ` [Qemu-devel] " Jamie Lokier
2009-11-20 7:33 ` Stefan Weil
2009-11-20 14:55 ` Jamie Lokier
2009-11-20 18:26 ` Stefan Weil
2009-11-20 16:58 ` Sebastian Herbszt
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B0639E7.40509@mail.berlios.de \
--to=weil@mail.berlios.de \
--cc=av1474@comtv.ru \
--cc=herbszt@gmx.de \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.