Re: [PATCH v2] spice: move client_migrate_info command to ui/

[Philippe Mathieu-Daudé]

On 20/4/23 14:37, Juan Quintela wrote:
> It has nothing to do with migration, except for the "migrate" in the
> name of the command.  Move it with the rest of the ui commands.
>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
>
> ---
>
> I would claim that it should be only compiled with CONFIG_SPICE, but
> I left that to spice maintainers.
>
> Gerd, what do you think?
> Should I take this patch over my tree?
>
> Based-on: Message-Id: <20230420121038.25167-1-quintela@redhat.com>
> [PATCH v3] migration: move migration_global_dump() to migration-hmp-cmds.c
>
> ---
>   qapi/migration.json            | 28 ----------------------------
>   qapi/ui.json                   | 28 ++++++++++++++++++++++++++++
>   migration/migration-hmp-cmds.c | 18 ------------------
>   migration/migration.c          | 30 ------------------------------
>   ui/ui-hmp-cmds.c               | 17 +++++++++++++++++
>   ui/ui-qmp-cmds.c               | 29 +++++++++++++++++++++++++++++
>   6 files changed, 74 insertions(+), 76 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>