Re: [PATCH] meson: Propagate gnutls dependency

[Peter Maydell]

On Mon, 4 Jan 2021 at 14:40, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> On 04/01/21 14:21, Peter Maydell wrote:
> >> The rest of QEMU should only ever be using QEMU's TLS abstractions
> >> and not directly be tied to GNUTLS. So ideally the gnutls flags
> >> should only ever be added in the crypto/meson.build, and anything
> >> which depends on that should then get the flags indirectly.
> > Unfortunately include/crypto/tlscreds.h leaks this implementation
> > detail
>
> That is not a problem.  As Daniel said, anything depending on crypto can
> still get the gnutls flags _indirectly_.
>
> In my proposed solution to the bug you'd get that via
>
>      io_ss.add(crypto, qom)
>      libio = static_library(..., dependencies: io_ss.dependencies())

Does this work recursively? For instance monitor/qmp-cmds.c
needs the gnutls CFLAGS because:
 * qmp-cmds.c includes ui/vnc.h
 * ui/vnc.h includes include/crypto/tlssession.h
 * include/crypto/tlssession.h includes gnutls.h

I don't see anything in monitor/meson.build that says "qmp-cmds.c
needs whatever ui's dependencies are".

thanks
-- PMM