From 62c979ee9c7b972d9841085de3fcd91a7d162ee7 Mon Sep 17 00:00:00 2001
From: David Levine <levinedl@acm.org>
Date: Sun, 20 Oct 2019 11:14:46 -0400
Subject: [PATCH 1/1] Updated documentation and comments about sendmail/pipe.

---
 man/mh-tailor.man |  9 ++++++---
 uip/post.c        | 17 ++++++++++-------
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/man/mh-tailor.man b/man/mh-tailor.man
index 0b0d2eff..894df14b 100644
--- a/man/mh-tailor.man
+++ b/man/mh-tailor.man
@@ -67,9 +67,11 @@ but feeds the message directly to it, using
 This replaces the old, undocumented
 .B spost
 mechanism and retains some of its limitations, such as lack of
-support for the
-.B \-whom
-switch and \*(lqDcc:\*(rq header field.
+support for the \*(lqDcc:\*(rq header field, as described in the
+.B send
+and
+.B post
+manual pages.
 .RE
 .PP
 .BR localname :
@@ -313,6 +315,7 @@ None
 .SH "SEE ALSO"
 .IR mh\-mts (8),
 .IR post (8)
+.IR send (8)
 .SH DEFAULTS
 As listed above.  The path of the mail transport configuration file can
 be changed with the MHMTSCONF environment variable and augmented with the
diff --git a/uip/post.c b/uip/post.c
index cf2ccf83..246f5670 100644
--- a/uip/post.c
+++ b/uip/post.c
@@ -760,7 +760,12 @@ main (int argc, char **argv)
 
     /* If we are doing a "whom" check */
     if (whomsw) {
-	/* This won't work with MTS_SENDMAIL_PIPE. */
+        /* Note that this will call sendmail directly using SMTP, just like
+           sendmail/smtp, even if the sendmail/pipe mts is specified.
+           It doesn't use sendmail -bv, which spost(8) used to do, because:
+             Notice: -bv may give misleading output for non-privileged user
+           and because -bv works differently with postfix.
+           And see the BUGS section of the whom(1) man page re. -check. */
         verify_all_addresses (1, eai, envelope, oauth_flag, auth_svc);
 	done (0);
     }
@@ -769,10 +774,9 @@ main (int argc, char **argv)
 	make_bcc_file (dashstuff);
 	if (msgflags & MVIS) {
 	    if (sm_mts != MTS_SENDMAIL_PIPE) {
-		/* It would be nice to have support to call
-		   verify_all_addresses with MTS_SENDMAIL_PIPE, but
-		   that might require running sendmail as root.  Note
-		   that spost didn't verify addresses. */
+                /* Addresses aren't verified here with sendmail/pipe because
+                   spost didn't.  They could be because verify_all_address()
+                   uses SMTP, see above. */
 		verify_all_addresses (verbose, eai, envelope, oauth_flag,
                                       auth_svc);
 	    }
@@ -1026,8 +1030,7 @@ putfmt (char *name, char *str, int *eai, FILE *out)
 			   aliases and put them in Bcc:, but then
 			   they'd have the Blind-Carbon-Copy
 			   indication. */
-			die(			       "blind lists not compatible with"
-			       " sendmail/pipe");
+			die("blind lists not compatible with sendmail/pipe");
 		    }
 
 		    grp = true;
-- 
2.56.0