From: Pierrick Bouvier <pierrick.bouvier@linaro.org>
To: qemu-devel@nongnu.org
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Pierrick Bouvier" <pierrick.bouvier@linaro.org>,
"Alexandre Iooss" <erdnaxe@crans.org>,
"Daniel P. Berrangé" <berrange@redhat.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Thomas Huth" <thuth@redhat.com>,
"Mahmoud Mandour" <ma.mandourr@gmail.com>
Subject: [PATCH v3 0/3] build contrib/plugins using meson
Date: Wed, 23 Oct 2024 14:28:09 -0700 [thread overview]
Message-ID: <20241023212812.1376972-1-pierrick.bouvier@linaro.org> (raw)
Contrib plugins have been built out of tree so far, thanks to a Makefile.
However, it is quite inconvenient for maintenance, as we may break them,
especially for specific architectures.
First patches are fixing warnings for existing plugins, then we add meson
support, and finally, we remove Makefile for contrib/plugins.
Based on the proposal of Anton Kochkov on associated gitlab issue.
Solves: https://gitlab.com/qemu-project/qemu/-/issues/1710
Plugins are now deactivated by default on 32-bits hosts (since cf2a78), so we
can enable with meson without worrying of warnings when building plugins for 32
bits.
Removed patches for individual plugins.
v2:
- removed warnings fix for 32 bits as they were incorrect. They are not needed
anymore as plugins are deprecated for 32 bits hosts.
v3:
- do not build lockstep on windows (it uses sys/socket.h)
- build new plugins recently added
- fix warning in cflow plugin
- tested it builds correctly on windows/macos/linux
Pierrick Bouvier (3):
contrib/plugins/cflow: fix warning
meson: build contrib/plugins with meson
contrib/plugins: remove Makefile for contrib/plugins
configure | 18 --------
Makefile | 10 -----
meson.build | 4 ++
contrib/plugins/cflow.c | 6 +--
contrib/plugins/Makefile | 87 -------------------------------------
contrib/plugins/meson.build | 28 ++++++++++++
6 files changed, 35 insertions(+), 118 deletions(-)
delete mode 100644 contrib/plugins/Makefile
create mode 100644 contrib/plugins/meson.build
--
2.39.5
next reply other threads:[~2024-10-23 21:29 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-23 21:28 Pierrick Bouvier [this message]
2024-10-23 21:28 ` [PATCH v3 1/3] contrib/plugins/cflow: fix warning Pierrick Bouvier
2024-10-23 21:28 ` [PATCH v3 2/3] meson: build contrib/plugins with meson Pierrick Bouvier
2024-10-23 21:28 ` [PATCH v3 3/3] contrib/plugins: remove Makefile for contrib/plugins Pierrick Bouvier
2024-10-24 10:15 ` [PATCH v3 0/3] build contrib/plugins using meson Alex Bennée
2024-11-04 22:29 ` 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=20241023212812.1376972-1-pierrick.bouvier@linaro.org \
--to=pierrick.bouvier@linaro.org \
--cc=alex.bennee@linaro.org \
--cc=berrange@redhat.com \
--cc=erdnaxe@crans.org \
--cc=ma.mandourr@gmail.com \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).