Re: [Qemu-devel] [PATCH ] qemu-help: add category headlines

[Markus Armbruster]

Marcel Apfelbaum <address@hidden> writes:

> On Thu, 2013-08-22 at 09:34 -0600, Eric Blake wrote:
>> On 08/22/2013 08:13 AM, Markus Armbruster wrote:
>> 
>> >>> +++ b/qdev-monitor.c
>> >>> @@ -156,6 +156,8 @@ static void
>> >>> qdev_print_category_devices(DeviceCategory category)
>> >>>      DeviceClass *dc;
>> >>>      GSList *list, *curr;
>> >>>  
>> >>> +    error_printf("%s devices:\n", qdev_category_get_name(category));
>> >>
>> >> Why is that an error? Shouldn't it go to stdout?
>> > 
>> > Output of -device help has always gone to stderr, and that has always
>> > annoyed me.  Just not enough to fix it.
>> 
>> Back when libvirt scraped -help output, changing it to use stdout would
>> be an incompatible change.  But now that libvirt uses QMP, I would also
>> welcome a change to use stdout (but also fall in the category of "not
>> enough of a bother for ME to fix it).
> It really bothers me. I need to use the help (being new to qemu ...)
> and I am always forgetting "2>&1".
>
> Anyone can think of a reason for not sending the help to stdout?   

Post a patch and find out ;)