make: fixes to -t option

The -t option (in general) causes targets to be touched instead of
having build commands run to create them.  There were two problems.

The flag variable 'ssilent' in docmds was too small (uint8_t) to
contain the value of 'dotouch' (uint32_t).  Truncation of the
value resulted in build commands being echoed when they shouldn't
have been.

The POSIX specification is unclear as to how build commands with
a '+' prefix interact with touch.  The rationale indicates that
this feature was imported from GNU make, so the behaviour has been
made to match what it does:  if a '+' build command is run the
target is not touched.

The code has been rearranged to move the call to touch() up into
docmds().

Adds 48 bytes.

1 files changed, 26 insertions, 0 deletions

diff --git a/testsuite/make.tests b/testsuite/make.tests
index 33e730602..c2b62532a 100755
--- a/testsuite/make.tests
+++ b/testsuite/make.tests
@@ -2,6 +2,7 @@
 
 . ./testing.sh
 unset MAKEFLAGS
+rm -rf make.tempdir
 
 # testing "test name" "command" "expected result" "file input" "stdin"
 
@@ -116,6 +117,31 @@ baz:
         @:
 '
 
+# The -t option touches files that are out-of-date unless the target
+# has no commands or they're already up-to-date.
+mkdir make.tempdir && cd make.tempdir || exit 1
+touch baz
+testing "make check -t option" \
+        "make -t -f - 2>/dev/null" "touch bar\n" "" '
+all: foo bar baz
+foo:
+bar:
+        @echo bar
+baz:
+        @echo baz
+'
+cd .. || exit 1; rm -rf make.tempdir 2>/dev/null
+
+# Build commands with a '+' prefix are executed even with the -t option.
+mkdir make.tempdir && cd make.tempdir || exit 1
+testing "make execute build command with + prefix and -t" \
+        "make -t -f - 2>/dev/null" "OK\n" "" '
+all: bar
+bar:
+        @+echo OK
+'
+cd .. || exit 1; rm -rf make.tempdir 2>/dev/null
+
 # A macro created using ::= remembers it's of type immediate-expansion.
 # Immediate expansion also occurs when += is used to append to such a macro.
 testing "make appending to immediate-expansion macro" \