Module Name:    src
Committed By:   rillig
Date:           Sun Aug 16 17:58:48 UTC 2020

Modified Files:
        src/usr.bin/make/unit-tests: Makefile

Log Message:
make(1): clean up cleaning of test files

The variables CLEANFILES and CLEANDIRS are not supposed to be
user-settable, thus use a simple "=" instead of "+=".

Since CLEANDIRS is always set, there is no point in making the rm
conditional.


To generate a diff of this commit:
cvs rdiff -u -r1.94 -r1.95 src/usr.bin/make/unit-tests/Makefile

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/usr.bin/make/unit-tests/Makefile
diff -u src/usr.bin/make/unit-tests/Makefile:1.94 src/usr.bin/make/unit-tests/Makefile:1.95
--- src/usr.bin/make/unit-tests/Makefile:1.94	Sun Aug 16 12:30:45 2020
+++ src/usr.bin/make/unit-tests/Makefile	Sun Aug 16 17:58:48 2020
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.94 2020/08/16 12:30:45 rillig Exp $
+# $NetBSD: Makefile,v 1.95 2020/08/16 17:58:48 rillig Exp $
 #
 # Unit tests for make(1)
 #
@@ -345,16 +345,14 @@ OUTFILES=	${TESTS:=.out}
 
 all: ${OUTFILES}
 
-CLEANFILES+=		*.rawout *.out *.status *.tmp *.core *.tmp
+CLEANFILES=		*.rawout *.out *.status *.tmp *.core *.tmp
 CLEANFILES+=		obj*.[och] lib*.a	# posix1.mk
 CLEANFILES+=		issue* .[ab]*		# suffixes.mk
-CLEANRECURSIVE+=	dir dummy		# posix1.mk
+CLEANDIRS=		dir dummy		# posix1.mk
 
 clean:
 	rm -f ${CLEANFILES}
-.if !empty(CLEANRECURSIVE)
-	rm -rf ${CLEANRECURSIVE}
-.endif
+	rm -rf ${CLEANDIRS}
 
 TEST_MAKE?=	${.MAKE}
 TOOL_SED?=	sed

Reply via email to