[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error
From: |
Hollis Blanchard |
Subject: |
Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error |
Date: |
Thu, 15 Oct 2009 14:18:31 -0700 |
On Thu, 2009-10-15 at 15:57 -0500, Anthony Liguori wrote:
> >>> Aside from that, we should certainly be able to change the pretty text,
> >>> for example, to provide additional clarification to the user. The
> >>> machine-interpreted code, on the other hand, wouldn't change.
> >>>
> >> How do you plan to do it in the design you're proposing? If you
> >> use the text from qemu_error() to build the user and protocol
> >> outputs, then you can't change it.
> >>
> >
> > Today it looks like this:
> > C: open host USB device foo
> > S: error 404, host USB device foo is already open
> >
> > Tomorrow it could look like this:
> > C: open host USB device foo
> > S: error 404, host USB device foo is already open by PID 27
> >
>
> What's tough about this sort of error handling is that it's not
> conducive to localization. It's not unusual for the server to have a
> different locale than the client so you really need the client to be
> able to translate error messages into meaningful messages in the client
> locale.
It's hard enough to keep localized strings updated for every release
within a single code tree. Beyond that, you expect that every *client*
will update its localized strings for every *server* release? If you
agree that is an unrealistic expectation, then even a "highly
structured" reporting mechanism won't help.
On the other hand, if you truly want to invest the energy required to
localize qemu (the server), I can imagine setting the per-client locale
as a monitor command.
--
Hollis Blanchard
IBM Linux Technology Center
- [Qemu-devel] Re: [PATCH 5/9] monitor: QError support, (continued)
- [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/13
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Hollis Blanchard, 2009/10/14
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Hollis Blanchard, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Hollis Blanchard, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Hollis Blanchard, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Anthony Liguori, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error,
Hollis Blanchard <=
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Anthony Liguori, 2009/10/15
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Hollis Blanchard, 2009/10/15
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Paolo Bonzini, 2009/10/16
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/16
- Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Daniel P. Berrange, 2009/10/19
- Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Luiz Capitulino, 2009/10/19
- Re: [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Daniel P. Berrange, 2009/10/19
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Anthony Liguori, 2009/10/16
- Re: [Qemu-devel] [PATCH 6/9] QError: Add qdev not found error, Jamie Lokier, 2009/10/18
- [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error, Paolo Bonzini, 2009/10/18