[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] tests: add rule to compile many objects u
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] tests: add rule to compile many objects used by tests |
Date: |
Mon, 19 Jun 2017 11:56:56 +0100 |
User-agent: |
mu4e 0.9.19; emacs 25.2.50.3 |
Philippe Mathieu-Daudé <address@hidden> writes:
> all those objects can get compiled simultaneously
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> tests/Makefile.include | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/tests/Makefile.include b/tests/Makefile.include
> index f42f3dfa72..3773f9d8d2 100644
> --- a/tests/Makefile.include
> +++ b/tests/Makefile.include
> @@ -804,6 +804,7 @@ check-help:
> @echo " make check-qapi-schema Run QAPI schema tests"
> @echo " make check-block Run block tests"
> @echo " make check-report.html Generates an HTML test report"
> + @echo " make check-compile Compile objects used by tests"
> @echo " make check-clean Clean the tests"
> @echo
> @echo "Please note that HTML reports do not regenerate if the unit
> tests"
> @@ -887,7 +888,7 @@ check-tests/qapi-schema/doc-good.texi:
> tests/qapi-schema/doc-good.test.texi
>
> # Consolidated targets
>
> -.PHONY: check-qapi-schema check-qtest check-unit check check-clean
> +.PHONY: check-qapi-schema check-qtest check-unit check check-clean
> check-compile
> check-qapi-schema: $(patsubst %,check-%, $(check-qapi-schema-y))
> check-tests/qapi-schema/doc-good.texi
> check-qtest: $(patsubst %,check-qtest-%, $(QTEST_TARGETS))
> check-unit: $(patsubst %,check-%, $(check-unit-y))
> @@ -897,7 +898,10 @@ check-clean:
> $(MAKE) -C tests/tcg clean
> rm -rf $(check-unit-y) tests/*.o $(QEMU_IOTESTS_HELPERS-y)
> rm -rf $(sort $(foreach target,$(SYSEMU_TARGET_LIST),
> $(check-qtest-$(target)-y)) $(check-qtest-generic-y))
> -
> +check-compile: $(check-qtest-generic-y) $(check-unit-y) $(test-util-obj-y)
> $(qtest-obj-y) $(test-io-obj-y) $(libqos-virtio-obj-y) $(libqos-pc-obj-y)
> $(chardev-obj-y) $(QEMU_IOTESTS_HELPERS-y)
> +ifeq ($(CONFIG_SOFTMMU),y)
> +check-compile: $(sort $(foreach target,$(SYSEMU_TARGET_LIST),
> $(check-qtest-$(target)-y)))
> +endif
> clean: check-clean
>
> # Build the help program automatically
Something has gone wrong here:
11:54 address@hidden taken:111, git:review/travis-speedup,
[/home/alex/lsrc/qemu/qemu.git]> make check-compile
CHK version_gen.h
LINK tests/ac97-test
tests/ac97-test.o: In function `main':
/home/alex/lsrc/qemu/qemu.git/tests/ac97-test.c:23: undefined reference to
`qtest_add_func'
tests/ac97-test.o: In function `qtest_start':
/home/alex/lsrc/qemu/qemu.git/tests/libqtest.h:521: undefined reference to
`qtest_init'
/home/alex/lsrc/qemu/qemu.git/tests/libqtest.h:521: undefined reference to
`global_qtest'
tests/ac97-test.o: In function `qtest_end':
/home/alex/lsrc/qemu/qemu.git/tests/libqtest.h:532: undefined reference to
`global_qtest'
/home/alex/lsrc/qemu/qemu.git/tests/libqtest.h:532: undefined reference to
`qtest_quit'
/home/alex/lsrc/qemu/qemu.git/tests/libqtest.h:533: undefined reference to
`global_qtest'
collect2: error: ld returned 1 exit status
/home/alex/lsrc/qemu/qemu.git/rules.mak:121: recipe for target
'tests/ac97-test' failed
make: *** [tests/ac97-test] Error 1
Yet "make check" completes without issue.
Also I would expect the check: target to have check-compile as one of
its pre-requisites otherwise this target will bit rot.
--
Alex Bennée
- [Qemu-devel] [PATCH 00/11] travis: try to reduce failures, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 01/11] travis: build using all available cpus, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 02/11] travis: split building/testing to have finer elapsed time, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 03/11] travis: use yes/no variable to enable/disable tests, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 04/11] tests: add rule to compile many objects used by tests, Philippe Mathieu-Daudé, 2017/06/16
- Re: [Qemu-devel] [PATCH 04/11] tests: add rule to compile many objects used by tests,
Alex Bennée <=
- [Qemu-devel] [PATCH 06/11] travis: retry if llvm.org timeouts, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 05/11] travis: build tests objects in parallel, then run tests sequentially, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 07/11] travis: install more library dependencies, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 09/11] scripts/run-coverity-scan: Script to run Coverity Scan build, Philippe Mathieu-Daudé, 2017/06/16
- [Qemu-devel] [PATCH 08/11] travis: install more library dependencies, Philippe Mathieu-Daudé, 2017/06/16