[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] qemu-iotests: fix test case 059
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] qemu-iotests: fix test case 059 |
Date: |
Wed, 25 Sep 2013 08:58:10 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 09/24 14:49, Stefan Hajnoczi wrote:
> On Tue, Sep 24, 2013 at 10:49:33AM +0800, Fam Zheng wrote:
> > Since commit "block: Error parameter for open functions", error output
> > is more verbose. Update test case output file to follow the change.
>
> This doesn't explain the "Formatting ..." trailing space change.
>
> If this has changed, please split the change into a separate patch, with
> an explanation of when the space was introduced, and fix the other test
> case output files:
>
> $ git grep '^Formatting' | grep '[^ ]$'
> 017.out:Formatting 'TEST_DIR/t.IMGFMT.base', fmt=IMGFMT size=6442450944
> 018.out:Formatting 'TEST_DIR/t.IMGFMT.base', fmt=IMGFMT size=6442450944
> 048.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=1073741824
> 059.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864
> 059.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864
> 059.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=67108864
> 063.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=4194304
> 063.out:Formatting 'TEST_DIR/t.IMGFMT', fmt=IMGFMT size=2097152
Not sure how it's introduced, it is actually ignored when comparing because we
have "-w" for diff command. I'll drop the spaces here and send another
revision, I'd like focus this series for fixing vmdk and leave the space clean
up for another series (if necessary).
Fam
[Qemu-devel] [PATCH v3 0/2] Fix and add test into 059., Fam Zheng, 2013/09/25
[Qemu-devel] [PATCH v4 0/2] Fix and add test into 059., Fam Zheng, 2013/09/25