[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] vnc: add a more descriptive e

From: Markus Armbruster
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] vnc: add a more descriptive error message
Date: Fri, 06 Jul 2012 09:31:45 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux)

Michael Tokarev <address@hidden> writes:

> On 06.07.2012 06:42, Amos Kong wrote:
>> On 30/06/12 10:02, address@hidden wrote:
>>> From: Amos Kong<address@hidden>
>>> Currently qemu outputs some low-level error in qemu-sockets.c
>>> when failed to start vnc server.
>>> eg. 'getaddrinfo(,5902): Name or service not known'
>>> Some libvirt users could not know what's happened with this
>>> unclear error message. This patch added a more descriptive
>>> error message.

Only libvirt users?  Really?

> Gyus, please, pretty PLEASE stop doing things like this.
> Amos, your patch does TWO things.  One is to clarify error
> message as correctly stated in your description, and second
> is to change the code to do exit(1) if this message is
> generated.  So, please, a) add the second fact to the
> description, and b) mention why it is needed.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]