Re: [PATCH v2 02/11] plugins: factorize plugin dependencies and library details

[Pierrick Bouvier]

On 1/16/26 4:22 AM, Daniel P. Berrangé wrote:

On Fri, Jan 02, 2026 at 01:47:15PM -0800, Pierrick Bouvier wrote:

Signed-off-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>
---
  meson.build                   |  2 +-
  contrib/plugins/meson.build   | 13 ++-----------
  plugins/meson.build           | 15 ++++++++++++++-
  tests/tcg/plugins/meson.build | 13 ++-----------
  4 files changed, 19 insertions(+), 24 deletions(-)

diff --git a/meson.build b/meson.build
index db87358d62d..3d6c6c702d0 100644
--- a/meson.build
+++ b/meson.build
@@ -4499,7 +4499,7 @@ if get_option('plugins')
    if host_os == 'windows'
      # On windows, we want to deliver the qemu_plugin_api.lib file in the qemu 
installer,
      # so that plugin authors can compile against it.
-    install_data(win32_qemu_plugin_api_lib, install_dir: 'lib')
+    install_data(win32_qemu_plugin_api, install_dir: 'lib')
    endif
  endif

diff --git a/contrib/plugins/meson.build b/contrib/plugins/meson.build

index 6f72b2ce0c9..8f9f0257ee5 100644
--- a/contrib/plugins/meson.build
+++ b/contrib/plugins/meson.build
@@ -9,17 +9,8 @@ endif
  t = []
  if get_option('plugins')
    foreach i : contrib_plugins
-    if host_os == 'windows'
-      t += shared_module(i, files(i + '.c') + '../../plugins/win32_linker.c',
-                        include_directories: '../../include/qemu',
-                        link_depends: [win32_qemu_plugin_api_lib],
-                        link_args: win32_qemu_plugin_api_link_flags,
-                        dependencies: glib)
-    else
-      t += shared_module(i, files(i + '.c'),
-                        include_directories: '../../include/qemu',
-                        dependencies: glib)
-    endif
+    t += shared_module(i, files(i + '.c'),
+                      dependencies: plugins_deps)

Nit-pick - under-indented by 1 space. The orignal code had the
same bug, but lets not preserve it.

diff --git a/tests/tcg/plugins/meson.build b/tests/tcg/plugins/meson.build
index a6e78438510..c58f2e382ae 100644
--- a/tests/tcg/plugins/meson.build
+++ b/tests/tcg/plugins/meson.build
@@ -1,17 +1,8 @@
  t = []
  if get_option('plugins')
    foreach i : ['bb', 'discons', 'empty', 'inline', 'insn', 'mem', 'reset', 
'syscall', 'patch']
-    if host_os == 'windows'
-      t += shared_module(i, files(i + '.c') + 
'../../../plugins/win32_linker.c',
-                        include_directories: '../../../include/qemu',
-                        link_depends: [win32_qemu_plugin_api_lib],
-                        link_args: win32_qemu_plugin_api_link_flags,
-                        dependencies: glib)
-    else
-      t += shared_module(i, files(i + '.c'),
-                        include_directories: '../../../include/qemu',
-                        dependencies: glib)
-    endif
+    t += shared_module(i, files(i + '.c'),
+                      dependencies: plugins_deps)

Same under-indent by 1.

With the indents fixed:

   Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>

Patch 3 already fixes the indent.

With regards,
Daniel