From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Cc: Mark Cave-Ayland <mark.caveayland@nutanix.com>,
Cornelia Huck <cohuck@redhat.com>,
Peter Maydell <peter.maydell@linaro.org>,
Richard Henderson <richard.henderson@linaro.org>,
qemu-devel@nongnu.org
Subject: Re: [PULL 00/21] Plugins patches for 2026-01-29
Date: Mon, 2 Feb 2026 17:34:48 +0000 [thread overview]
Message-ID: <aYDgONbUOWdUi7SO@redhat.com> (raw)
In-Reply-To: <bd097ad0-d18e-42fc-9746-db4d448b71fa@linaro.org>
On Mon, Feb 02, 2026 at 09:30:31AM -0800, Pierrick Bouvier wrote:
> On 2/2/26 8:55 AM, Mark Cave-Ayland wrote:
> > On 02/02/2026 15:42, Cornelia Huck wrote:
> >
> > > On Mon, Feb 02 2026, Peter Maydell <peter.maydell@linaro.org> wrote:
> > >
> > > > On Mon, 2 Feb 2026 at 04:04, Richard Henderson
> > > > <richard.henderson@linaro.org> wrote:
> > > > >
> > > > > On 1/30/26 04:41, Pierrick Bouvier wrote:
> > > > > > On 1/29/26 9:37 AM, Pierrick Bouvier wrote:
> > > > > > > contrib/plugins: add empty cpp plugin
> > > >
> > > > Hi; this fails to build for me on Linux (Ubuntu 24.04) with clang:
> > > >
> > > > In file included from ../../contrib/plugins/cpp.cpp:33:
> > > > /usr/bin/../lib/gcc/x86_64-linux-gnu/14/../../../../include/c++/14/coroutine:38:3:
> > > > error: "the <coroutine> header requires -fcoroutines"
> > > > 38 | # error "the <coroutine> header requires -fcoroutines"
> > > > | ^
> > > > 1 error generated.
> > >
> > > Fails for me as well on Fedora 42 (x86_64) with gcc:
> > >
> > > In file included from ../qemu/contrib/plugins/cpp.cpp:33:
> > > /usr/include/c++/15/coroutine:43:3: error: #error "the <coroutine> header requires -fcoroutines"
> > > 43 | # error "the <coroutine> header requires -fcoroutines"
> > > | ^~~~~
> > >
> > > (It works for me on an aarch64 RHEL 9 system.)
> >
> > Is this with a clean build? I had exactly the same error message
> > earlier, but I was able to fix it by destroying my build/ directory,
> > then re-running configure followed again by make.
> >
>
> Thanks for the reports everyone.
>
> I tried it on ubuntu LTS and didn't encounter this problem.
> What is happening is that meson unfortunately caches default_options set,
> including the C++ standard version in our case.
> Thus, you indeed need to clean and rebuild once.
>
> The solution for this would be to not use default_options and add the flag
> to qemu_cflags instead. It's the same limitation if we change the default
> warning level, or any other option set with default_options.
If that's correct as the root cause, then it feels like this is a meson bug
that ought to be reported upstream. Any meson.build default_options changes
that affect the compiler behaviour should invalidate cache in the same way
as when we modify qemu_cflags.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2026-02-02 17:35 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-29 17:37 [PULL 00/21] Plugins patches for 2026-01-29 Pierrick Bouvier
2026-01-29 17:37 ` [PULL 01/21] contrib/plugins/hotblocks: Correctly free sorted counts list Pierrick Bouvier
2026-01-29 17:37 ` [PULL 02/21] contrib/plugins/hotblocks: Fix off by one error in iteration of sorted blocks Pierrick Bouvier
2026-01-29 17:37 ` [PULL 03/21] contrib/plugins/hotblocks: Print uint64_t with PRIu64 rather than PRId64 Pierrick Bouvier
2026-01-29 17:37 ` [PULL 04/21] docs/about/emulation: Add documentation for hotblocks plugin arguments Pierrick Bouvier
2026-01-29 17:37 ` [PULL 05/21] contrib/plugins/hotblocks: Allow limit to be set as a command line argument Pierrick Bouvier
2026-01-29 17:37 ` [PULL 06/21] linux-user: move user/syscall-trace.h to linux-user/syscall.c Pierrick Bouvier
2026-01-29 17:37 ` [PULL 07/21] linux-user: add plugin API to filter syscalls Pierrick Bouvier
2026-01-29 17:37 ` [PULL 08/21] tcg tests: add a test to verify the syscall filter plugin API Pierrick Bouvier
2026-01-29 17:37 ` [PULL 09/21] plugins: return bool from register r/w API Pierrick Bouvier
2026-01-29 17:37 ` [PULL 10/21] plugins: move win32_linker.c file to plugins directory Pierrick Bouvier
2026-01-29 17:37 ` [PULL 11/21] plugins: factorize plugin dependencies and library details Pierrick Bouvier
2026-01-29 17:37 ` [PULL 12/21] plugins: use complete filename for defining plugins sources Pierrick Bouvier
2026-01-29 17:37 ` [PULL 13/21] plugins: define plugin API symbols as extern "C" when compiling in C++ Pierrick Bouvier
2026-01-29 17:38 ` [PULL 14/21] tests/tcg/plugins/mem.c: remove dependency on qemu headers Pierrick Bouvier
2026-01-29 17:38 ` [PULL 15/21] plugins: move qemu-plugin.h to include/plugins/ Pierrick Bouvier
2026-01-29 17:38 ` [PULL 16/21] meson: fix supported compiler arguments in other languages than C Pierrick Bouvier
2026-01-29 17:38 ` [PULL 17/21] meson: enable cpp (optionally) for plugins Pierrick Bouvier
2026-01-29 17:38 ` [PULL 18/21] qga/vss-win32: fix clang warning with C++20 Pierrick Bouvier
2026-01-29 17:38 ` [PULL 19/21] meson: update C++ standard to C++23 Pierrick Bouvier
2026-01-29 17:38 ` [PULL 20/21] contrib/plugins: add empty cpp plugin Pierrick Bouvier
2026-01-29 17:38 ` [PULL 21/21] plugins: reduce source conflicts in plugins list Pierrick Bouvier
2026-01-29 17:41 ` [PULL 00/21] Plugins patches for 2026-01-29 Pierrick Bouvier
2026-02-02 4:03 ` Richard Henderson
2026-02-02 13:26 ` Peter Maydell
2026-02-02 15:42 ` Cornelia Huck
2026-02-02 16:55 ` Mark Cave-Ayland
2026-02-02 17:25 ` Cornelia Huck
2026-02-02 17:30 ` Pierrick Bouvier
2026-02-02 17:34 ` Daniel P. Berrangé [this message]
2026-02-02 17:45 ` Pierrick Bouvier
2026-02-02 17:49 ` Daniel P. Berrangé
2026-02-02 17:51 ` Pierrick Bouvier
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aYDgONbUOWdUi7SO@redhat.com \
--to=berrange@redhat.com \
--cc=cohuck@redhat.com \
--cc=mark.caveayland@nutanix.com \
--cc=peter.maydell@linaro.org \
--cc=pierrick.bouvier@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.