tests/Makefile.include: Let "make clean" remove the TCG tests, too

[PATCH] tests/Makefile.include: Let "make clean" remove the TCG tests, too

Thomas Huth posted 1 patch 3 years, 11 months ago

Download mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220301085900.1443232-1-thuth@redhat.com

tests/Makefile.include | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[PATCH] tests/Makefile.include: Let "make clean" remove the TCG tests, too

Posted by Thomas Huth 3 years, 11 months ago

"make clean" should clear all binaries that have been built, but so
far it left the TCG tests still in place. Let's make sure that they
are now removed, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/Makefile.include | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/Makefile.include b/tests/Makefile.include
index e7153c8e91..7a932caf91 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -162,6 +162,6 @@ check-build: run-ninja
 check-clean:
 	rm -rf $(TESTS_VENV_DIR) $(TESTS_RESULTS_DIR)
 
-clean: check-clean
+clean: check-clean clean-tcg
 
 endif
-- 
2.27.0

Re: [PATCH] tests/Makefile.include: Let "make clean" remove the TCG tests, too

Posted by Richard Henderson 3 years, 11 months ago

On 2/28/22 22:59, Thomas Huth wrote:
> "make clean" should clear all binaries that have been built, but so
> far it left the TCG tests still in place. Let's make sure that they
> are now removed, too.
> 
> Signed-off-by: Thomas Huth<thuth@redhat.com>
> ---
>   tests/Makefile.include | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH] tests/Makefile.include: Let "make clean" remove the TCG tests, too

Posted by Alex Bennée 3 years, 11 months ago

Thomas Huth <thuth@redhat.com> writes:

> "make clean" should clear all binaries that have been built, but so
> far it left the TCG tests still in place. Let's make sure that they
> are now removed, too.

Queued to for-7.0/misc-bits, thanks.

-- 
Alex Bennée