Re: [Qemu-devel] [PULL 28/48] tests: Add ipoctal232 qtest

["Andreas Färber" <afaerber@suse.de>]

Am 10.02.2014 19:36, schrieb Andreas Färber:
> Cc: Alberto Garcia <agarcia@igalia.com>
> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
> Signed-off-by: Andreas Färber <afaerber@suse.de>
> ---
>  tests/Makefile          |  7 +++++++
>  tests/ipoctal232-test.c | 33 +++++++++++++++++++++++++++++++++
>  2 files changed, 40 insertions(+)
>  create mode 100644 tests/ipoctal232-test.c
> 
> diff --git a/tests/Makefile b/tests/Makefile
> index 1a8cc72..59e71c2 100644
> --- a/tests/Makefile
> +++ b/tests/Makefile
> @@ -65,6 +65,10 @@ check-block-$(CONFIG_POSIX) += tests/qemu-iotests-quick.sh
>  # All QTests for now are POSIX-only, but the dependencies are
>  # really in libqtest, not in the testcases themselves.
>  
> +gcov-files-ipack-y += hw/char/ipack.c
> +check-qtest-ipack-y += tests/ipoctal232-test$(EXESUF)
> +gcov-files-ipack-y += hw/char/ipoctal232.c
> +
>  gcov-files-virtio-y += hw/virtio/virtio.c
>  check-qtest-virtio-y += tests/virtio-net-test$(EXESUF)
>  gcov-files-virtio-y += hw/net/virtio-net.c
> @@ -82,6 +86,8 @@ check-qtest-pci-y += $(check-qtest-virtio-y)
>  gcov-files-pci-y += $(gcov-files-virtio-y) hw/virtio/virtio-pci.c
>  check-qtest-pci-y += tests/tpci200-test$(EXESUF)
>  gcov-files-pci-y += hw/char/tpci200.c
> +check-qtest-pci-y += $(check-qtest-ipack-y)
> +gcov-files-ipack-y += $(gcov-files-ipack-y) hw/char/tpci200.c

Copy & paste error! Needs:

diff --git a/tests/Makefile b/tests/Makefile
index f09c9b8..584fe4e 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -89,7 +89,7 @@ gcov-files-pci-y += $(gcov-files-virtio-y)
hw/virtio/virtio-pci.c
 check-qtest-pci-y += tests/tpci200-test$(EXESUF)
 gcov-files-pci-y += hw/char/tpci200.c
 check-qtest-pci-y += $(check-qtest-ipack-y)
-gcov-files-ipack-y += $(gcov-files-ipack-y) hw/char/tpci200.c
+gcov-files-pci-y += $(gcov-files-ipack-y) hw/char/tpci200.c

 check-qtest-i386-y = tests/endianness-test$(EXESUF)
 check-qtest-i386-y += tests/fdc-test$(EXESUF)

Fixing for v2.

Andreas

>  
>  check-qtest-i386-y = tests/endianness-test$(EXESUF)
>  check-qtest-i386-y += tests/fdc-test$(EXESUF)
> @@ -229,6 +235,7 @@ tests/vmxnet3-test$(EXESUF): tests/vmxnet3-test.o
>  tests/ne2000-test$(EXESUF): tests/ne2000-test.o
>  tests/virtio-net-test$(EXESUF): tests/virtio-net-test.o
>  tests/tpci200-test$(EXESUF): tests/tpci200-test.o
> +tests/ipoctal232-test$(EXESUF): tests/ipoctal232-test.o
>  tests/qom-test$(EXESUF): tests/qom-test.o
>  tests/blockdev-test$(EXESUF): tests/blockdev-test.o $(libqos-pc-obj-y)
>  tests/qdev-monitor-test$(EXESUF): tests/qdev-monitor-test.o $(libqos-pc-obj-y)

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg