From: NHOrus <jy6x2b32pie9@yahoo.com>
Date: Sat, 1 Jul 2017 07:51:13 +0000 (+0300)
Subject: Fix make debug check
X-Git-Tag: takebird~164
X-Git-Url: https://jxself.org/git/?a=commitdiff_plain;h=b0705ba0bbbee2cc1648c5b9adbe0e147e1dfea9;p=open-adventure.git

Fix make debug check
---

diff --git a/Makefile b/Makefile
index e9c0de3..43f1f95 100644
--- a/Makefile
+++ b/Makefile
@@ -27,6 +27,7 @@
 VERS=$(shell sed -n <NEWS '/^[0-9]/s/:.*//p' | head -1)
 
 .PHONY: debug indent release refresh dist linty html clean
+.PHONY: check coverage
 
 CC?=gcc
 CCFLAGS+=-std=c99 -D_DEFAULT_SOURCE -DVERSION=\"$(VERS)\" -Wpedantic -O2
@@ -46,7 +47,7 @@ SOURCES=$(OBJS:.o=.c) advent.h adventure.yaml Makefile control linenoise/linenoi
 advent:	$(OBJS) linenoise.o dungeon.o
 	$(CC) $(CCFLAGS) $(DBX) -o advent $(OBJS) dungeon.o linenoise.o $(LDFLAGS) $(LIBS)
 
-main.o:	 	linenoise-gitmodule advent.h dungeon.h
+main.o:	 	advent.h dungeon.h linenoise.o
 
 init.o:	 	advent.h dungeon.h
 
@@ -56,7 +57,7 @@ score.o:	advent.h dungeon.h
 
 misc.o:		advent.h dungeon.h
 
-cheat.o:	linenoise-gitmodule advent.h dungeon.h
+cheat.o:	advent.h dungeon.h linenoise.o
 
 saveresume.o:	advent.h dungeon.h
 
@@ -66,13 +67,14 @@ dungeon.o:	dungeon.c dungeon.h
 dungeon.c dungeon.h: make_dungeon.py adventure.yaml
 	python3 make_dungeon.py
 
-linenoise-gitmodule: 
-	test -s linenoise/linenoise.h || { echo "\nlinenoise not present. Try: \n\n\tgit submodule update --recursive --remote --init\n"; exit 1; }
-
-linenoise.o:	linenoise-gitmodule linenoise/linenoise.h
+linenoise.o: linenoise/linenoise.h
+	@test -s linenoise/linenoise.h || { echo -e "linenoise not present. Try:\n\
+		git submodule update --recursive --remote --init"; exit 1; }
 	$(CC) -c linenoise/linenoise.c
 
-linenoise-dbg:	linenoise-gitmodule linenoise/linenoise.h
+linenoise-dbg: linenoise/linenoise.h
+	@test -s linenoise/linenoise.h || { echo -e "linenoise not present. Try:\n\
+		git submodule update --recursive --remote --init"; exit 1; }
 	$(CC) $(CCFLAGS) -c linenoise/linenoise.c
 
 clean:
diff --git a/tests/Makefile b/tests/Makefile
index d2ccb14..afc7fdf 100644
--- a/tests/Makefile
+++ b/tests/Makefile
@@ -8,7 +8,13 @@ PATH := $(realpath ..):$(realpath .):${PATH}
 # Defeat annoying behavior under Mac OS X - builtin echo doesn't do -n
 ECHO := /bin/echo
 
-check: regress
+# Find all *.log entries to test
+TESTLOADS := $(shell ls -1 *.log | sed '/.log/s///' | sort)
+
+.PHONY: check coverage clean testlist listcheck savegames buildregress
+.PHONY: savecheck regress
+
+check: savecheck regress
 	@echo "=== No diff output is good news."
 	@-advent -x 2>/dev/null	# Get usage message into coverage tests
 	@-advent -l /dev/null <pitfall.log >/dev/null
@@ -30,6 +36,7 @@ listcheck:
 	    if ( head -3 $$f | grep -q '^ *##' ); then :; else echo "$$f needs a description"; fi; \
 	done
 
+# Generate bogus savegames.
 savegames:
 	$(ECHO) "cheat: Generate save file with -1000 deaths"
 	../cheat -d -1000 -o cheat_numdie.adv > /tmp/cheat_numdie
@@ -39,9 +46,8 @@ savegames:
 	../cheat -s -1000 -o thousand_saves.adv > /tmp/cheat_1000saves
 	rm -f /tmp/cheat*
 
-# General regression testing of commands and output; look at the *.log and
-# corresponding *.chk files to see which tests this runs.
-TESTLOADS := $(shell ls -1 *.log | sed '/.log/s///' | sort)
+
+# Rebuild characterizing tests
 buildregress: savegames
 	../cheat -s -1000 -o thousand_saves.adv > /tmp/regress1000saves
 	@for file in $(TESTLOADS); do \
@@ -50,7 +56,8 @@ buildregress: savegames
 	    advent $$OPTS <$${file}.log >$${file}.chk 2>&1 || exit 1; \
 	done; \
 	rm -f scratch.tmp
-regress: savegames
+
+savecheck: savegames
 	$(ECHO) "TEST cheat: Bogus option for save file generation"
 	../cheat -QqQ 2> /tmp/coverage_cheat_batopt | true
 	$(ECHO) "TEST cheat: Fail to save because we omit -o"
@@ -63,13 +70,17 @@ regress: savegames
 	advent -l / < pitfall.log > /tmp/coverage_advent_logfail 2>&1 || exit 1
 	$(ECHO) "TEST advent: Test -r with valid input"
 	advent -r thousand_saves.adv < pitfall.log > /tmp/coverage_advent_readfail 2>&1 || exit 1
+	rm -f scratch.tmp /tmp/regress$$$$ /tmp/coverage*
+
+# General regression testing of commands and output; look at the *.log and
+# corresponding *.chk files to see which tests this runs.
+regress:
 	@for file in $(TESTLOADS); do \
 	    $(ECHO) -n "  $${file} "; grep '##' $${file}.log  || echo ' ## (no description)'; \
 	    OPTS=`sed -n /#options:/s///p <$${file}.log`; \
 	    if advent $$OPTS < $${file}.log >/tmp/regress$$$$ 2>&1; \
 	    then diff --text -u $${file}.chk /tmp/regress$$$$ || exit 1; \
 	    else echo "*** Nonzero return status on $${file}!"; exit 1; fi \
-	done; \
-	rm -f scratch.tmp /tmp/regress$$$$ /tmp/coverage*
+	done;
 
 # end