[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/33] Block patches
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL 00/33] Block patches |
Date: |
Wed, 28 May 2014 10:55:26 +0100 |
On 28 May 2014 10:52, Stefan Hajnoczi <address@hidden> wrote:
> On Tue, May 27, 2014 at 06:15:29PM +0100, Peter Maydell wrote:
>> On 23 May 2014 16:41, Stefan Hajnoczi <address@hidden> wrote:
>> > The following changes since commit
>> > 178ac111bca16c08a79b2609ebdc75197bea976a:
>> >
>> > Merge remote-tracking branch 'remotes/qmp-unstable/queue/qmp' into
>> > staging (2014-05-22 19:04:49 +0100)
>> >
>> > are available in the git repository at:
>> >
>> >
>> > git://github.com/stefanha/qemu.git tags/block-pull-request
>> >
>> > for you to fetch changes up to 1c53366b9589b5438336dce86d6ffea35bf80b15:
>> >
>> > block/sheepdog: Don't use qerror_report() (2014-05-23 17:39:26 +0200)
>>
>> This pullreq seems to introduce extra noise into "make check":
>>
>> GTESTER tests/test-qemu-opts
>> There is no option group 'unknown'
>>
>> Either that should be a test failure or it shouldn't be
>> outputting anything.
>
> Here is a fix. Please review and I'll squash it into a PULL v2.
>
> From a22e5a88c0d11ca6171d349b44b20dd3a1c78c87 Mon Sep 17 00:00:00 2001
> From: Stefan Hajnoczi <address@hidden>
> Date: Wed, 28 May 2014 11:43:13 +0200
> Subject: [PATCH] QemuOpt: use qemu_find_opts_err() to avoid output on stderr
> in tests
>
> qemu_find_opts() calls error_report() instead of propagating the Error
> object. It is undesirable to clutter test case output with error
> messages from a passing test.
>
> Use qemu_find_opts_err() to avoid the output on stderr.
>
> Cc: Leandro Dorileo <address@hidden>
> Cc: Peter Maydell <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Thanks for fixing this -- it's a small thing but I'm trying to keep the
'make check' output from getting cluttered with random output waffle.
-- PMM
- [Qemu-devel] [PULL 28/33] block/sheepdog: Propagate errors through sd_prealloc(), (continued)
- [Qemu-devel] [PULL 28/33] block/sheepdog: Propagate errors through sd_prealloc(), Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 29/33] block/sheepdog: Propagate errors through do_sd_create(), Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 30/33] block/sheepdog: Propagate errors through find_vdi_name(), Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 31/33] block/sheepdog: Propagate errors to open and create methods, Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 32/33] block/sheepdog: Fix silent sd_open(), sd_create() failures, Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 33/33] block/sheepdog: Don't use qerror_report(), Stefan Hajnoczi, 2014/05/23
- [Qemu-devel] [PULL 22/33] block/ssh: Propagate errors through connect_to_ssh(), Stefan Hajnoczi, 2014/05/23
- Re: [Qemu-devel] [PULL 00/33] Block patches, Peter Maydell, 2014/05/27