[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 3/3] iotests: Use -qmp-pretty in 067
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v4 3/3] iotests: Use -qmp-pretty in 067 |
Date: |
Thu, 20 Nov 2014 19:39:03 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 17.11.2014 um 13:31 hat Max Reitz geschrieben:
> 067 invokes query-block, resulting in a reference output with really
> long lines (which may pose a problem in email patches and always poses a
> problem when the output changes, because it is hard to see what has
> actually changed). Use -qmp-pretty to mitigate this issue.
>
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---
> tests/qemu-iotests/067 | 2 +-
> tests/qemu-iotests/067.out | 779
> +++++++++++++++++++++++++++++++++++++++++----
> 2 files changed, 723 insertions(+), 58 deletions(-)
Hm, when I tried to rebase my query-block patch that adds cache
information, I noticed that I get a trailing space on every line of the
JSON dump, which isn't there in this patch. So 'cp 067.out.bad 067.out'
ended up changing every single line. :-/
Did you postprocess the reference output or do really get it without
trailing whitespace?
Kevin
Re: [Qemu-devel] [PATCH v4 0/3] chardev: Add -qmp-pretty, Kevin Wolf, 2014/11/20
Re: [Qemu-devel] [PATCH v4 0/3] chardev: Add -qmp-pretty, Markus Armbruster, 2014/11/28