Re: [Qemu-devel] [PATCH] tests: Fix test 049 fallout from improved HMP error messages
Kevin Wolf writes: > Am 23.09.2015 um 01:15 hat Eric Blake geschrieben: >> Commit 50b7b000 improved HMP error messages, but forgot to update >> qemu-iotests to match. >> >> Reported-by: Kevin Wolf >> Signed-off-by: Eric Blake > > Thanks, applied to the block branch. > >> I'm not sure if this should go through Markus' "error" tree, >> since that's where the break was introduced, or through a >> "block" tree since it touches qemu-iotests. You copied both maintainers, so they can figure it out. Good move. > Doesn't really matter as long as someone takes it. And if both took > it, I think git should be clever enough to figure it out without merge > conflicts. Since you took it, I won't. Thanks!
Re: [Qemu-devel] [PATCH] tests: Fix test 049 fallout from improved HMP error messages
Am 23.09.2015 um 01:15 hat Eric Blake geschrieben: > Commit 50b7b000 improved HMP error messages, but forgot to update > qemu-iotests to match. > > Reported-by: Kevin Wolf > Signed-off-by: Eric Blake Thanks, applied to the block branch. > I'm not sure if this should go through Markus' "error" tree, > since that's where the break was introduced, or through a > "block" tree since it touches qemu-iotests. Doesn't really matter as long as someone takes it. And if both took it, I think git should be clever enough to figure it out without merge conflicts. Kevin
Re: [Qemu-devel] [PATCH] tests: Fix test 049 fallout from improved HMP error messages
On 09/22/2015 07:15 PM, Eric Blake wrote: > Commit 50b7b000 improved HMP error messages, but forgot to update > qemu-iotests to match. > > Reported-by: Kevin Wolf > Signed-off-by: Eric Blake > --- > > I'm not sure if this should go through Markus' "error" tree, > since that's where the break was introduced, or through a > "block" tree since it touches qemu-iotests. > > And now that I have a recipe for running qemu-iotests, I should > have no excuse for repeating this type of mistake :) > > tests/qemu-iotests/049.out | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/qemu-iotests/049.out b/tests/qemu-iotests/049.out > index 0425ae0..a2b6703 100644 > --- a/tests/qemu-iotests/049.out > +++ b/tests/qemu-iotests/049.out > @@ -118,6 +118,7 @@ qemu-img: kilobytes, megabytes, gigabytes, terabytes, > petabytes and exabytes. > > qemu-img create -f qcow2 -o size=foobar TEST_DIR/t.qcow2 > qemu-img: Parameter 'size' expects a size > +You may use k, M, G or T suffixes for kilobytes, megabytes, gigabytes and > terabytes. > qemu-img: TEST_DIR/t.qcow2: Invalid options for file format 'qcow2' > > == Check correct interpretation of suffixes for cluster size == > Grazie, Reviewed-by: John Snow