[src/trunk]: src/usr.bin/make make(1): fix archive test

[rillig <rillig%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/30cced0b4af3
branches:  trunk
changeset: 1012905:30cced0b4af3
user:      rillig <rillig%NetBSD.org@localhost>
date:      Sat Aug 15 01:49:07 2020 +0000

description:
make(1): fix archive test

Even though it is commented out in this Makefile, it is still run by
atf, on purpose.  This test is broken on FreeBSD, but on NetBSD it
should still be run.

Without removing the archive first, it could be left over from a
previous test failure and thus be newer than all its members, leading to
${.OODATE} being empty.

diffstat:

 usr.bin/make/Makefile              |  4 ++--
 usr.bin/make/unit-tests/Makefile   |  6 ++++--
 usr.bin/make/unit-tests/archive.mk |  5 +++--
 3 files changed, 9 insertions(+), 6 deletions(-)

diffs (62 lines):

diff -r 00f415747b42 -r 30cced0b4af3 usr.bin/make/Makefile
--- a/usr.bin/make/Makefile     Sat Aug 15 01:27:22 2020 +0000
+++ b/usr.bin/make/Makefile     Sat Aug 15 01:49:07 2020 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: Makefile,v 1.88 2020/08/12 19:21:05 rillig Exp $
+#      $NetBSD: Makefile,v 1.89 2020/08/15 01:49:07 rillig Exp $
 #      @(#)Makefile    5.2 (Berkeley) 12/28/90
 
 PROG=  make
@@ -173,7 +173,7 @@
        sed -i 's,^\([^:]*\): *[0-9]*:,\1: ,' *.gcov
 .endif
 
-accept: .MAKE
+accept sync-mi: .MAKE
        cd ${.CURDIR}/unit-tests && ${.MAKE} ${.TARGET}
 
 retest:
diff -r 00f415747b42 -r 30cced0b4af3 usr.bin/make/unit-tests/Makefile
--- a/usr.bin/make/unit-tests/Makefile  Sat Aug 15 01:27:22 2020 +0000
+++ b/usr.bin/make/unit-tests/Makefile  Sat Aug 15 01:49:07 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.91 2020/08/09 16:32:28 rillig Exp $
+# $NetBSD: Makefile,v 1.92 2020/08/15 01:49:07 rillig Exp $
 #
 # Unit tests for make(1)
 #
@@ -33,7 +33,9 @@
 
 # Each test is in a sub-makefile.
 # Keep the list sorted.
-TESTS+=                # archive       # broken on FreeBSD
+# Any test that is commented out must be ignored in
+# src/tests/usr.bin/make/t_make.sh as well.
+TESTS+=                # archive       # broken on FreeBSD, enabled in t_make.sh
 TESTS+=                cmdline
 TESTS+=                comment
 TESTS+=                cond-late
diff -r 00f415747b42 -r 30cced0b4af3 usr.bin/make/unit-tests/archive.mk
--- a/usr.bin/make/unit-tests/archive.mk        Sat Aug 15 01:27:22 2020 +0000
+++ b/usr.bin/make/unit-tests/archive.mk        Sat Aug 15 01:49:07 2020 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: archive.mk,v 1.1 2020/07/27 18:51:03 rillig Exp $
+# $NetBSD: archive.mk,v 1.2 2020/08/15 01:49:07 rillig Exp $
 #
 # Very basic demonstration of handling archives, based on the description
 # in PSD.doc/tutorial.ms.
@@ -13,6 +13,7 @@
 RUN?=          @set -eu;
 
 all:
+       ${RUN} ${MAKE_CMD} remove-archive
        ${RUN} ${MAKE_CMD} create-archive
        ${RUN} ${MAKE_CMD} list-archive
        ${RUN} ${MAKE_CMD} depend-on-existing-member
@@ -33,5 +34,5 @@
 depend-on-nonexistent-member: ${ARCHIVE}(nonexistent.mk)
        ${RUN} echo $@
 
-remove-archive: ${ARCHIVE}
+remove-archive:
        rm ${ARCHIVE}