diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2021-07-28 15:34:43 +0400 |
|---|---|---|
| committer | Marc-André Lureau <marcandre.lureau@redhat.com> | 2021-08-05 16:15:33 +0400 |
| commit | 64195b0d365712c05ae922b7ff22cf136aac1400 (patch) | |
| tree | 10be11affd1eecbd41fb3f9e0104a39a39e8aa8c /chardev/char-console.c | |
| parent | 733ba020846ccd21d832f2e9b62387a86c5ab8f1 (diff) | |
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 'chardev/char-console.c')
0 files changed, 0 insertions, 0 deletions