Updated documentation and comments about sendmail/pipe.

diff --git a/man/mh-tailor.man b/man/mh-tailor.man
index 0b0d2ef..894df14 100644 (file)
--- 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 cf2ccf8..246f567 100644 (file)
--- 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;