From c4259ff3defcac0d8f8075fd99884eef22e5d966 Mon Sep 17 00:00:00 2001 From: Thomas Miedema Date: Tue, 26 Apr 2016 15:12:05 +0200 Subject: Testsuite: make CLEANUP=1 the default (#9758) Also move the `cleanup` setting from `default_testopts` to `config`. The `cleanup` setting is the same for all tests, hence it belongs in `config`. Reviewed by: austin Differential Revision: https://phabricator.haskell.org/D2148 --- testsuite/mk/test.mk | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) (limited to 'testsuite/mk') diff --git a/testsuite/mk/test.mk b/testsuite/mk/test.mk index 6d9a4c24cf..8198efbc8c 100644 --- a/testsuite/mk/test.mk +++ b/testsuite/mk/test.mk @@ -189,6 +189,14 @@ ifeq "$(SKIP_PERF_TESTS)" "YES" RUNTEST_OPTS += --skip-perf-tests endif +ifeq "$(CLEANUP)" "0" +RUNTEST_OPTS += -e cleanup=False +else ifeq "$(CLEANUP)" "NO" +RUNTEST_OPTS += -e cleanup=False +else +RUNTEST_OPTS += -e cleanup=True +endif + ifneq "$(CLEAN_ONLY)" "" RUNTEST_OPTS += -e clean_only=True else @@ -207,7 +215,6 @@ RUNTEST_OPTS += \ -e 'config.os="$(TargetOS_CPP)"' \ -e 'config.arch="$(TargetARCH_CPP)"' \ -e 'config.wordsize="$(WORDSIZE)"' \ - -e 'default_testopts.cleanup="$(CLEANUP)"' \ -e 'config.timeout=int($(TIMEOUT)) or config.timeout' \ -e 'config.exeext="$(exeext)"' \ -e 'config.top="$(TOP_ABS)"' @@ -326,15 +333,15 @@ list_broken: # From # https://www.gnu.org/software/make/manual/html_node/Variables_002fRecursion.html: # -# "The ‘-j’ option is a special case (see Parallel Execution). If you set -# it to some numeric value ‘N’ and your operating system supports it (most -# any UNIX system will; others typically won’t), the parent make and all the -# sub-makes will communicate to ensure that there are only ‘N’ jobs running +# "The '-j' option is a special case (see Parallel Execution). If you set +# it to some numeric value 'N' and your operating system supports it (most +# any UNIX system will; others typically won't), the parent make and all the +# sub-makes will communicate to ensure that there are only 'N' jobs running # at the same time between them all." # # In our scenario, the user will actually see the following warning [2]: # -# ‘warning: jobserver unavailable: using -j1. Add `+' to parent make rule.’ +# 'warning: jobserver unavailable: using -j1. Add '+' to parent make rule.' # # The problem is that topmake and submake don't know about eachother, since # python is in between. To let them communicate, we have to use the '+' -- cgit v1.2.1