qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Stefan Weil <sw@weilnetz.de>
To: "Andreas Färber" <afaerber@suse.de>
Cc: blauwirbel@gmail.com, Anthony Liguori <aliguori@us.ibm.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2] w32: Build windows and console executables
Date: Fri, 13 Jan 2012 22:57:34 +0100	[thread overview]
Message-ID: <4F10A8CE.3050807@weilnetz.de> (raw)
In-Reply-To: <4F10A2B0.5050902@suse.de>

Am 13.01.2012 22:31, schrieb Andreas Färber:
> Am 13.01.2012 22:13, schrieb Stefan Weil:
>> Am 22.12.2011 11:20, schrieb Stefan Weil:
>>> System emulation executables with SDL are typically windows
>>> executables. Sometimes console executables are more useful,
>>> so create both variants if linker option -mwindows was detected.
>>>
>>> v2:
>>> This version uses QEMU_PROGW / QEMU_PROG instead of QEMU_PROG /
>>> QEMU_PROGC.
>>>
>>> Signed-off-by: Stefan Weil<sw@weilnetz.de>
>>> ---
>>> Makefile.target | 16 +++++++++++++++-
>>> 1 files changed, 15 insertions(+), 1 deletions(-)
>
>> Ping? All open questions were solved, so this patch can be committed.
>
> No objection to this patch, just raising the question whether we might
> want to compile code differently for the two targets? For example, does
> -nographic make sense for the *w.exe?
>
> Andreas

Yes, it makes sense.

It's possible to run a *w.exe with stdin/stdout redirected
to a pipe. The msys rxvt terminal works like that and allows
-nographics.

And even without such tricks, the commonly used SDL
implementations redirect stdout to a file, so it is possible
to run a *w.exe with -nographic and use the file (for example
in automated tests which boot a Linux kernel).

So I see no reason to compile both executables differently.

Having executables which are binary compatible has also
advantages for installers (compressing twice the same binary
data should be easy).

Regards,
Stefan

      reply	other threads:[~2012-01-13 21:57 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-22 10:20 [Qemu-devel] [PATCH v2] w32: Build windows and console executables Stefan Weil
2011-12-23  2:31 ` TeLeMan
2011-12-23 13:47   ` Stefan Weil
2011-12-26  2:14     ` TeLeMan
2012-01-13 21:13 ` Stefan Weil
2012-01-13 21:31   ` Andreas Färber
2012-01-13 21:57     ` Stefan Weil [this message]

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=4F10A8CE.3050807@weilnetz.de \
    --to=sw@weilnetz.de \
    --cc=afaerber@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=blauwirbel@gmail.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).