summary refs log tree commit diff stats
path: root/scripts/qapi/expr.py
diff options
context:
space:
mode:
authorMarc-André Lureau <marcandre.lureau@redhat.com>2021-07-28 15:34:43 +0400
committerMarc-André Lureau <marcandre.lureau@redhat.com>2021-08-05 16:15:33 +0400
commit64195b0d365712c05ae922b7ff22cf136aac1400 (patch)
tree10be11affd1eecbd41fb3f9e0104a39a39e8aa8c /scripts/qapi/expr.py
parent733ba020846ccd21d832f2e9b62387a86c5ab8f1 (diff)
downloadfocaccia-qemu-64195b0d365712c05ae922b7ff22cf136aac1400.tar.gz
focaccia-qemu-64195b0d365712c05ae922b7ff22cf136aac1400.zip
chardev: give some context on chardev-add error
Description from Daniel P. Berrangé:
> The original code reported:
>
>  "attempt to add duplicate property 'char2' to object (type 'container')"
>
> Since adding yank support, the current code reports
>
>  "duplicate yank instance"
>
> With this patch applied it now reports:
>
>  "Failed to add chardev 'char2': duplicate yank instance"
>
> This is marginally better, but still not great, not that the original
> error was great either.
>
> It would be nice if we could report
>
>   "chardev with id 'char2' already exists"

Related to:
https://bugzilla.redhat.com/show_bug.cgi?id=1984721

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions