Re: [PATCH v4 0/3] plugins: generate list of symbols automatically

[Pierrick Bouvier]

On 11/12/24 13:26, Pierrick Bouvier wrote:
> Now that meson build for plugins was merged, we can cleanup another part with
> the symbols file.
> It has to be kept in sync between the header (qemu-plugin.h) and the symbols
> file. This has proved to be error prone and tedious.
>
> We solve this by generating this list from header directly using a python
> script. Dependencies are triggered correctly, and the file is regenerated if the
> header is modified, as expected.
>
> v4:
> fix second patch to apply cleanly on master (was applied from a previous series
> to enable windows clang build before).
>
> Pierrick Bouvier (3):
>    plugins: add missing export for qemu_plugin_num_vcpus
>    plugins: detect qemu plugin API symbols from header
>    plugins: eradicate qemu-plugins.symbols static file
>
>   MAINTAINERS                    |  1 +
>   include/qemu/qemu-plugin.h     |  1 +
>   plugins/meson.build            | 12 +++++--
>   plugins/qemu-plugins.symbols   | 59 ----------------------------------
>   scripts/qemu-plugin-symbols.py | 45 ++++++++++++++++++++++++++
>   5 files changed, 56 insertions(+), 62 deletions(-)
>   delete mode 100644 plugins/qemu-plugins.symbols
>   create mode 100755 scripts/qemu-plugin-symbols.py

Gentle ping, to hopefully merge this in 9.2.