From e038f74b28f8a48e03e91bfe459a5af587a112fa Mon Sep 17 00:00:00 2001 From: Dianne Skoll Date: Tue, 7 Oct 2025 20:31:21 -0400 Subject: [PATCH] Document that INCLUDECMD's cmd has stdin < /dev/null Make testing yet a bit more verbose. --- man/remind.1.in | 3 +++ tests/test-rem | 3 ++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/man/remind.1.in b/man/remind.1.in index 0ccdaa42..6fd665d9 100644 --- a/man/remind.1.in +++ b/man/remind.1.in @@ -2446,6 +2446,9 @@ you can write a command that transforms them to \fBRemind\fR format and then Note that if RUN is disabled, then INCLUDECMD will fail with the error message "RUN disabled" .PP +\fBRemind\fR arranges so that when the command specified by INCLUDECMD is +executed, its standard input is opened to /dev/null. +.PP INCLUDECMD passes the rest of the line to \fBpopen\fR(3), meaning that the command is executed by the shell. As such, shell meta-characters may need escaping or arguments quoting, depending on what you're trying diff --git a/tests/test-rem b/tests/test-rem index 26c47203..0bdfa288 100644 --- a/tests/test-rem +++ b/tests/test-rem @@ -624,6 +624,7 @@ grep -F -v -a '$SysInclude' < $OUT > $OUT.1 && mv -f $OUT.1 $OUT RUNMAN=0 man man 2>&1 | grep -e --warnings > /dev/null 2>&1 if test "$?" = 0 ; then + echo "Checking syntax of man pages" RUNMAN=1 fi for i in ../man/*.1 ; do @@ -762,7 +763,7 @@ $REMIND ../tests/ret1.rem 7 June 2000 >> $OUT 2>&1 $REMIND -s ../tests/ret1.rem 1 June 2000 >> $OUT 2>&1 # Make sure all the include files are ok -echo "Checking files in ../include" +echo "Checking syntax of *.rem files in ../include" find ../include -type f -name '*.rem' | while read x; do $REMIND -du -n $x 1 Jan 2024 2>>$OUT 1>/dev/null; done # Test todos