[Qemu-devel] [PATCH] Work around stdio redirection by SDL on Windows

[Qemu-devel] [PATCH] Work around stdio redirection by SDL on Windows

[Sebastian Herbszt]

Work around no longer working console output with SDL on Windows.

Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>

--- v0.11.0-rc0-201-gd9c3231/vl.c.orig Sun Aug  9 11:52:47 2009
+++ v0.11.0-rc0-201-gd9c3231/vl.c Sun Aug  9 11:58:59 2009
@@ -4873,6 +4873,12 @@
     CPUState *env;
     int show_vnc_port = 0;
 
+/* Work around stdio redirection by SDL on Windows */
+#if defined (CONFIG_SDL) && defined (_WIN32)
+    freopen("CON", "wb", stdout);
+    freopen("CON", "wb", stderr);
+#endif
+
     qemu_cache_utils_init(envp);
 
     LIST_INIT (&vm_change_state_head);

[Qemu-devel] Re: [PATCH] Work around stdio redirection by SDL on Windows

[Filip Navara]

On Sun, Aug 9, 2009 at 2:06 PM, Sebastian Herbszt<herbszt@gmx.de> wrote:
> Work around no longer working console output with SDL on Windows.
>
> Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>
>
> --- v0.11.0-rc0-201-gd9c3231/vl.c.orig Sun Aug  9 11:52:47 2009
> +++ v0.11.0-rc0-201-gd9c3231/vl.c Sun Aug  9 11:58:59 2009
> @@ -4873,6 +4873,12 @@
>    CPUState *env;
>    int show_vnc_port = 0;
>
> +/* Work around stdio redirection by SDL on Windows */
> +#if defined (CONFIG_SDL) && defined (_WIN32)
> +    freopen("CON", "wb", stdout);
> +    freopen("CON", "wb", stderr);
> +#endif
> +

That will not work in RXVT, SSH consoles and for pipes unfortunately
:-/ Also it won't respect the redirections specified on command line,
which is something I often use, therefore I object to this patch. It's
better to use SDL compiled with the --enable-stdio-redirect=no option.

Best regards,
Filip Navara

[Qemu-devel] Re: [PATCH] Work around stdio redirection by SDL on Windows

[Sebastian Herbszt]

Filip Navara wrote:
>> Sebastian Herbszt wrote:
>> --- v0.11.0-rc0-201-gd9c3231/vl.c.orig Sun Aug 9 11:52:47 2009
>> +++ v0.11.0-rc0-201-gd9c3231/vl.c Sun Aug 9 11:58:59 2009
>> @@ -4873,6 +4873,12 @@
>> CPUState *env;
>> int show_vnc_port = 0;
>>
>> +/* Work around stdio redirection by SDL on Windows */
>> +#if defined (CONFIG_SDL) && defined (_WIN32)
>> + freopen("CON", "wb", stdout);
>> + freopen("CON", "wb", stderr);
>> +#endif
>> +
>
> That will not work in RXVT, SSH consoles and for pipes unfortunately
> :-/ Also it won't respect the redirections specified on command line,
> which is something I often use, therefore I object to this patch. It's
> better to use SDL compiled with the --enable-stdio-redirect=no option.

Too bad this work around is a no go. At least for me it's good enough to
make qemu usable again.

I hope the need for a custom SDL will at least get properly documented since
this is a big regression for Windows users.

- Sebastian