On Tue, Sep 20, 2022 at 3:04 PM Bin Meng <bmeng...@gmail.com> wrote:
> From: Bin Meng <bin.m...@windriver.com>
>
> On Windows the QEMU executable is created via CreateProcess() and
> IO redirection does not work, so don't bother adding IO redirection
> to the command line.
>
> Signed-off-by: Bin Meng <bin.m...@windriver.com>
> ---
>
> Changes in v2:
> - Change the place that sets IO redirection in the command line
>
hmm ok,
Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> tests/qtest/migration-test.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/tests/qtest/migration-test.c b/tests/qtest/migration-test.c
> index 9925691ead..c87afad9e8 100644
> --- a/tests/qtest/migration-test.c
> +++ b/tests/qtest/migration-test.c
> @@ -648,7 +648,16 @@ static int test_migrate_start(QTestState **from,
> QTestState **to,
> }
>
> if (!getenv("QTEST_LOG") && args->hide_stderr) {
> +#ifndef _WIN32
> ignore_stderr = "2>/dev/null";
> +#else
> + /*
> + * On Windows the QEMU executable is created via CreateProcess()
> and
> + * IO redirection does not work, so don't bother adding IO
> redirection
> + * to the command line.
> + */
> + ignore_stderr = "";
> +#endif
> } else {
> ignore_stderr = "";
> }
> --
> 2.34.1
>
>
>
--
Marc-André Lureau
