This a much improved version of my first attempt for this tool [1].

This:
    - replaces tests/qemucapabilitiesnumbering.c
        - including much simpler approach for programatic modification
          of the .replies files

    - adds 'dump' capability to simplify comparing two qemu versions or
      figuring out query strings for qmp schema

[1]: [PATCH] scripts: QMP schema query string helper script

Peter Krempa (10):
  tests: qemucaps: Make JSON output identical to python's 'json.dump'
    method
  syntax-check: Exclude 'scripts/*.py' from
    'sc_prohibit_magic_number_exit'
  scripts: Add 'qemu-qmp-replies-tool' script for testing and modifying
    data for qemucapabilitiestest
  tests: Remove 'qemucapabilitiesnumbering' test
  util: json: Remove 'virJSONValueObjectReplaceValue'
  qemumonitortestutils: Unexport 'qemuMonitorTestProcessFileEntries'
  qemu-replies-tool: Add validation of known fields in
    'query-qmp-schema'
  qemu-replies-tool: Add mode to dump all QMP schema query strings
  qemu-replies-tool: Dump 'qom-list-types'
  qemu-replies-tool: Dump 'device-list-properties'

 build-aux/syntax-check.mk                     |   3 +
 scripts/meson.build                           |   1 +
 scripts/qemu-replies-tool.py                  | 557 ++++++++++++++++++
 src/libvirt_private.syms                      |   1 -
 src/util/virjson.c                            |  20 -
 src/util/virjson.h                            |   6 -
 tests/meson.build                             |  12 +-
 .../caps_4.2.0_aarch64.replies                |  42 +-
 .../caps_4.2.0_ppc64.replies                  |  42 +-
 .../caps_4.2.0_s390x.replies                  | 297 ++++------
 .../caps_4.2.0_x86_64.replies                 | 363 ++++--------
 .../caps_5.0.0_aarch64.replies                |  39 +-
 .../caps_5.0.0_ppc64.replies                  |  39 +-
 .../caps_5.0.0_riscv64.replies                |  42 +-
 .../caps_5.0.0_x86_64.replies                 | 372 ++++--------
 .../caps_5.1.0_sparc.replies                  |  42 +-
 .../caps_5.1.0_x86_64.replies                 | 291 +++------
 .../caps_5.2.0_aarch64.replies                |  39 +-
 .../caps_5.2.0_ppc64.replies                  |  39 +-
 .../caps_5.2.0_riscv64.replies                |  42 +-
 .../caps_5.2.0_s390x.replies                  | 303 ++++------
 .../caps_5.2.0_x86_64.replies                 | 291 +++------
 .../caps_6.0.0_aarch64.replies                |  39 +-
 .../caps_6.0.0_s390x.replies                  | 303 ++++------
 .../caps_6.0.0_x86_64.replies                 | 291 +++------
 .../caps_6.1.0_x86_64.replies                 | 297 ++++------
 .../caps_6.2.0_aarch64.replies                |  45 +-
 .../caps_6.2.0_ppc64.replies                  |  45 +-
 .../caps_6.2.0_x86_64.replies                 | 297 ++++------
 .../caps_7.0.0_aarch64+hvf.replies            |  45 +-
 .../caps_7.0.0_aarch64.replies                |  45 +-
 .../caps_7.0.0_ppc64.replies                  |  45 +-
 .../caps_7.0.0_x86_64.replies                 | 297 ++++------
 .../caps_7.1.0_ppc64.replies                  |  45 +-
 .../caps_7.1.0_x86_64.replies                 | 297 ++++------
 .../caps_7.2.0_ppc.replies                    |  45 +-
 .../caps_7.2.0_x86_64+hvf.replies             | 297 ++++------
 .../caps_7.2.0_x86_64.replies                 | 297 ++++------
 .../caps_8.0.0_riscv64.replies                |  36 +-
 .../caps_8.0.0_x86_64.replies                 | 297 ++++------
 .../caps_8.1.0_s390x.replies                  | 318 ++++------
 .../caps_8.1.0_x86_64.replies                 | 285 +++------
 .../caps_8.2.0_aarch64.replies                |  48 +-
 .../caps_8.2.0_x86_64.replies                 | 288 +++------
 .../caps_9.0.0_x86_64.replies                 | 288 +++------
 tests/qemucapabilitiesnumbering.c             | 245 --------
 tests/qemucapsprobemock.c                     |  28 +-
 tests/qemumonitortestutils.c                  |   9 +-
 tests/qemumonitortestutils.h                  |  13 -
 49 files changed, 2796 insertions(+), 4672 deletions(-)
 create mode 100755 scripts/qemu-replies-tool.py
 delete mode 100644 tests/qemucapabilitiesnumbering.c

-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org

On a Tuesday in 2024, Peter Krempa wrote:
>This a much improved version of my first attempt for this tool [1].
>
>This:
>    - replaces tests/qemucapabilitiesnumbering.c
>        - including much simpler approach for programatic modification
>          of the .replies files
>
>    - adds 'dump' capability to simplify comparing two qemu versions or
>      figuring out query strings for qmp schema
>
>[1]: [PATCH] scripts: QMP schema query string helper script
>
>Peter Krempa (10):
>  tests: qemucaps: Make JSON output identical to python's 'json.dump'
>    method
>  syntax-check: Exclude 'scripts/*.py' from
>    'sc_prohibit_magic_number_exit'
>  scripts: Add 'qemu-qmp-replies-tool' script for testing and modifying
>    data for qemucapabilitiestest
>  tests: Remove 'qemucapabilitiesnumbering' test
>  util: json: Remove 'virJSONValueObjectReplaceValue'
>  qemumonitortestutils: Unexport 'qemuMonitorTestProcessFileEntries'
>  qemu-replies-tool: Add validation of known fields in
>    'query-qmp-schema'
>  qemu-replies-tool: Add mode to dump all QMP schema query strings
>  qemu-replies-tool: Dump 'qom-list-types'
>  qemu-replies-tool: Dump 'device-list-properties'
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org