[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] tests/Makefile: Remove unused test-obj-y vari
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-trivial] [PATCH] tests/Makefile: Remove unused test-obj-y variable |
Date: |
Tue, 7 May 2019 14:10:09 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 06/05/19 00:18, Thomas Huth wrote:
> I recently noticed that test-obj-y contains a file called
> tests/check-block-qtest.o which simply does not belong to any .c
> file and thus wondered why this is not causing any trouble.
> Well, if I get the Makefile magic right, test-obj-y is not really
> used for anything - and "make check" still works fine if we simply
> remove it.
>
> Signed-off-by: Thomas Huth <address@hidden>
It is used to add -Itests to the command line (which refers to the build
directory). However, it is not needed because "-iquote $(@D)" already
sets this up in rules.mak.
Paolo
> ---
> tests/Makefile.include | 19 -------------------
> 1 file changed, 19 deletions(-)
>
> diff --git a/tests/Makefile.include b/tests/Makefile.include
> index 7c8b9c84b2..dfc4b7746f 100644
> --- a/tests/Makefile.include
> +++ b/tests/Makefile.include
> @@ -482,25 +482,6 @@ GENERATED_FILES += tests/test-qapi-types.h \
> tests/test-qapi-events-sub-sub-module.h \
> tests/test-qapi-introspect.h
>
> -test-obj-y = tests/check-qnum.o tests/check-qstring.o tests/check-qdict.o \
> - tests/check-qlist.o tests/check-qnull.o tests/check-qobject.o \
> - tests/check-qjson.o tests/check-qlit.o \
> - tests/check-block-qtest.o \
> - tests/test-coroutine.o tests/test-string-output-visitor.o \
> - tests/test-string-input-visitor.o tests/test-qobject-output-visitor.o \
> - tests/test-clone-visitor.o \
> - tests/test-qobject-input-visitor.o \
> - tests/test-qmp-cmds.o tests/test-visitor-serialization.o \
> - tests/test-x86-cpuid.o tests/test-mul64.o tests/test-int128.o \
> - tests/test-opts-visitor.o tests/test-qmp-event.o \
> - tests/rcutorture.o tests/test-rcu-list.o \
> - tests/test-rcu-simpleq.o \
> - tests/test-rcu-tailq.o \
> - tests/test-qdist.o tests/test-shift128.o \
> - tests/test-qht.o tests/qht-bench.o tests/test-qht-par.o \
> - tests/atomic_add-bench.o tests/atomic64-bench.o
> -
> -$(test-obj-y): QEMU_INCLUDES += -Itests
> QEMU_CFLAGS += -I$(SRC_PATH)/tests
>
>
>