Re: [PATCH 3/4] qtest: Improve error messages when property can not be set right now

[Thomas Huth]

On 12/10/2022 17.38, Markus Armbruster wrote:
> When you try to set qtest property "log" while the qtest object is
> active, the error message blames "insufficient permission":
>
>      $ qemu-system-x86_64 -S -display none -nodefaults -monitor stdio -chardev 
> socket,id=chrqt0,path=qtest.socket,server=on,wait=off -object 
> qtest,id=qt0,chardev=chrqt0,log=/dev/null
>      QEMU 7.1.50 monitor - type 'help' for more information
>      (qemu) qom-set /objects/qt0 log qtest.log
>      Error: Insufficient permission to perform this operation
>
> This implies it could work with "sufficient permission".  It can't.
> Change the error message to:
>
>      Error: Property 'log' can not be set now
Can it be set later? ... if not, that error message is almost as confusing 
as the original one. Maybe it's better to tell the users *when* they can set 
the property?
 Thomas