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
 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 :
 .RE
 .PP
 .BR localname :


@@ -313,6 +315,7 @@ None
 .SH "SEE ALSO"
 .IR mh\-mts (8),
 .IR post (8)
 .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
 .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) {
 
     /* 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);
     }
         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) {
        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);
            }
                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. */
                           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;
                    }
 
                    grp = true;