Re: [PATCH v2 06/25] python/qemu-ga-client: update instructions to newer CLI syntax

2021-12-16 Thread John Snow
On Thu, Dec 16, 2021 at 5:14 AM Vladimir Sementsov-Ogievskiy < vsement...@virtuozzo.com> wrote: > I had to search a bit through the history to check this ) > > The commit ccd3b3b8112b670f "qemu-option: warn for short-form boolean > options" may be noted here. > > And may be subject changed to "don

Re: [PATCH v2 06/25] python/qemu-ga-client: update instructions to newer CLI syntax

2021-12-16 Thread Daniel P . Berrangé
On Wed, Dec 15, 2021 at 02:39:20PM -0500, John Snow wrote: > Signed-off-by: John Snow > --- > python/qemu/qmp/qemu_ga_client.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Daniel P. Berrangé Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com

Re: [PATCH v2 06/25] python/qemu-ga-client: update instructions to newer CLI syntax

2021-12-16 Thread Vladimir Sementsov-Ogievskiy
I had to search a bit through the history to check this ) The commit ccd3b3b8112b670f "qemu-option: warn for short-form boolean options" may be noted here. And may be subject changed to "don't use deprecated syntax in comment" or something like this. server=on is not a *new* syntax I think. 1

[PATCH v2 06/25] python/qemu-ga-client: update instructions to newer CLI syntax

2021-12-15 Thread John Snow
Signed-off-by: John Snow --- python/qemu/qmp/qemu_ga_client.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/qemu/qmp/qemu_ga_client.py b/python/qemu/qmp/qemu_ga_client.py index 67ac0b4211..b3e1d98c9e 100644 --- a/python/qemu/qmp/qemu_ga_client.py +++ b/python/qemu/q