[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v8 27/27] Revert "configure: add --ninja option"
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v8 27/27] Revert "configure: add --ninja option" |
Date: |
Sun, 13 Sep 2020 16:08:32 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 13/09/20 00:44, Yonggang Luo wrote:
> This reverts commit 48328880fddf0145bdccc499160fb24dfabfbd41.
>
> The --ninja option doesn't need anymore because of upgrade meson to 0.55.2
> At that version we can use ninjatool
We might actually get rid of ninjatool before QEMU 5.2 goes out, if we
decide to make Ninja a mandatory build dependency. So we can hold on
patches 26 and 27. Thanks for testing though!
I'm also not sure about patch 16, since that's not my area, but Daniel
and Ed both reviewed it so that's okay.
Finally, instead of checking !_WIN32 it's better to check CONFIG_POSIX
or CONFIG_WIN32. That can be changed on commit though.
Everything else seems okay. I'll wait a couple days and queue the whole
bunch up to patch 25.
Paolo
- Re: [PATCH v8 20/27] tests: Fixes test-io-channel-file by mask only owner file state mask bits, (continued)
- [PATCH v8 21/27] tests: fix test-util-sockets.c, Yonggang Luo, 2020/09/12
- [PATCH v8 22/27] tests: Fixes test-qdev-global-props.c, Yonggang Luo, 2020/09/12
- [PATCH v8 23/27] rcu: fixes test-logging.c by call drain_call_rcu before rmdir_full, Yonggang Luo, 2020/09/12
- [PATCH v8 24/27] ci: Enable msys2 ci in cirrus, Yonggang Luo, 2020/09/12
- [PATCH v8 25/27] meson: upgrade meson for execute custom ninjatool under msys2 properly, Yonggang Luo, 2020/09/12
- [PATCH v8 26/27] meson: remove --ninja option in configure., Yonggang Luo, 2020/09/12
- [PATCH v8 27/27] Revert "configure: add --ninja option", Yonggang Luo, 2020/09/12
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option",
Paolo Bonzini <=
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option", Yonggang Luo, 2020/09/13
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option", Paolo Bonzini, 2020/09/14
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option", Yonggang Luo, 2020/09/14
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option", Thomas Huth, 2020/09/15
- Re: [PATCH v8 27/27] Revert "configure: add --ninja option", Yonggang Luo, 2020/09/15
Re: [PATCH v8 00/27] W32, W64 msys2/mingw patches, no-reply, 2020/09/12