From: Daniel P. Berrange Date: Tue, 16 Jan 2018 13:42:09 +0000 (+0000) Subject: qapi: remove '-q' arg to diff when comparing QAPI output X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=46ec4fcea95204a8e5bab9295cbfaa3606d78dc9;p=qemu.git qapi: remove '-q' arg to diff when comparing QAPI output When the qapi schema tests fail they merely print that the expected output didn't match the actual output. This is largely useless when trying diagnose what went wrong. Removing the '-q' arg to diff means that it is still silent on successful tests, but when it fails we'll see details of the incorrect output. Reviewed-by: Eric Blake Signed-off-by: Daniel P. Berrange Message-Id: <20180116134217.8725-7-berrange@redhat.com> Signed-off-by: Eduardo Habkost --- diff --git a/tests/Makefile.include b/tests/Makefile.include index ca82e0c0cc..3258b9c22a 100644 --- a/tests/Makefile.include +++ b/tests/Makefile.include @@ -917,10 +917,10 @@ $(patsubst %, check-%, $(check-qapi-schema-y)): check-%.json: $(SRC_PATH)/%.json $^ >$*.test.out 2>$*.test.err; \ echo $$? >$*.test.exit, \ "TEST","$*.out") - @diff -q $(SRC_PATH)/$*.out $*.test.out + @diff $(SRC_PATH)/$*.out $*.test.out @# Sanitize error messages (make them independent of build directory) - @perl -p -e 's|\Q$(SRC_PATH)\E/||g' $*.test.err | diff -q $(SRC_PATH)/$*.err - - @diff -q $(SRC_PATH)/$*.exit $*.test.exit + @perl -p -e 's|\Q$(SRC_PATH)\E/||g' $*.test.err | diff $(SRC_PATH)/$*.err - + @diff $(SRC_PATH)/$*.exit $*.test.exit .PHONY: check-tests/qapi-schema/doc-good.texi check-tests/qapi-schema/doc-good.texi: tests/qapi-schema/doc-good.test.texi