|
From: | Paolo Bonzini |
Subject: | Re: [PATCH 15/24] configure, meson: move libaio check to meson.build |
Date: | Fri, 8 Oct 2021 11:15:15 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.0 |
On 08/10/21 10:47, Marc-André Lureau wrote:
It looks silly but it works ("when: []" is always-true, just like in python all([]) is true). I'll remove the if_true in v2. and why not keep the one-liner version?: stub_ss.add(when: libaio, if_true: files('linux-aio.c'))
It would link libaio even in files that don't use it. The stub is used by those files that get util/async.c via libqemuutil, but not block/linux-aio.c.
This particular stub (and the equivalent io_uring one) is a bit silly though.
Someone should check exactly which files get which stub, maybe there are opportunities for simplifications. For example if a stub is only used by the unit tests, we might as well move more code to libqemuutil and remove the stub.
Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |