On Sun, Sep 16, 2018 at 12:25:47PM +0200, Sebastian Benoit wrote: > Jason McIntyre(j...@kerhand.co.uk) on 2018.09.16 07:17:24 +0100: > > On Sat, Sep 15, 2018 at 09:42:47PM -0400, Matt Schwartz wrote: > > > Damnit! I forgot one line. Please disregard the previous. Now all > > > lines of mailer.conf(5) should reflect that sendmail is no longer in > > > the default install. > > > > > > > i'm ok with this, though the spacing on your diff got mangled and it > > doesn;t apply. > > > > i'd be tempted to remove the text ", named > > /usr/local/libexec/sendmail/sendmail" > > > > anyone want to ok (or commit with my ok)? > > ok benno@ > > possible further improvements: > > * maybe we should mention that sendmail is a package that needs tobe > installed? I dont know if we do that in manpages usually? >
i thought about changing the comment (Execute the real sendmail...) from naming the path to naming the package, but i don;t like it. i still want to trim that comment, since speciying that path, then relisting it 5 times seems unneccessary. we could add some (separate) text about adding sendmail as a package. i'm not sure if that would be helpful or not. > * the BUGS section can go away i think, its wishfull thinking because > these program names are the defacto standard. > i'm ok with removing that idea, but the man point of the entry seems to be that having a need for this file is a bug in itself. shouldn;t we retain the removal of this system as a goal? jmc > > > jmc > > > > > Index: mailer.conf.5 > > > =================================================================== > > > RCS file: /cvs/src/usr.sbin/mailwrapper/mailer.conf.5,v > > > retrieving revision 1.17 > > > diff -u -p -u -r1.17 mailer.conf.5 > > > --- mailer.conf.5 8 Dec 2015 09:06:32 -0000 1.17 > > > +++ mailer.conf.5 16 Sep 2018 01:41:53 -0000 > > > @@ -78,14 +78,14 @@ This example shows how to invoke the tra > > > MTA suite in place of > > > .Xr smtpd 8 . > > > .Bd -literal > > > -# Execute the "real" sendmail program, named > > > /usr/libexec/sendmail/sendmail > > > -sendmail /usr/libexec/sendmail/sendmail > > > -send-mail /usr/libexec/sendmail/sendmail > > > -mailq /usr/libexec/sendmail/sendmail > > > -makemap /usr/libexec/sendmail/makemap > > > -newaliases /usr/libexec/sendmail/sendmail > > > -hoststat /usr/libexec/sendmail/sendmail > > > -purgestat /usr/libexec/sendmail/sendmail > > > +# Execute the "real" sendmail program, named > > > /usr/local/libexec/sendmail/sendmail > > > +sendmail /usr/local/libexec/sendmail/sendmail > > > +send-mail /usr/local/libexec/sendmail/sendmail > > > +mailq /usr/local/libexec/sendmail/sendmail > > > +makemap /usr/local/libexec/sendmail/makemap > > > +newaliases /usr/local/libexec/sendmail/sendmail > > > +hoststat /usr/local/libexec/sendmail/sendmail > > > +purgestat /usr/local/libexec/sendmail/sendmail > > > .Ed > > > .Sh SEE ALSO > > > .Xr mail 1 , > > > > > > On Sat, Sep 15, 2018 at 9:36 PM Matt Schwartz <matt.schwart...@gmail.com> > > > wrote: > > > > > > > > Quick correction to the mailer.conf(5) man page. Since sendmail is no > > > > longer in the default install, the man page has been corrected to > > > > reflect that the sendmail binary would be in > > > > /usr/local/libexec/sendmail instead of /usr/libexec/sendmail. > > > > > > > > Index: mailer.conf.5 > > > > =================================================================== > > > > RCS file: /cvs/src/usr.sbin/mailwrapper/mailer.conf.5,v > > > > retrieving revision 1.17 > > > > diff -u -p -u -r1.17 mailer.conf.5 > > > > --- mailer.conf.5 8 Dec 2015 09:06:32 -0000 1.17 > > > > +++ mailer.conf.5 16 Sep 2018 01:34:31 -0000 > > > > @@ -79,13 +79,13 @@ MTA suite in place of > > > > .Xr smtpd 8 . > > > > .Bd -literal > > > > # Execute the "real" sendmail program, named > > > > /usr/libexec/sendmail/sendmail > > > > -sendmail /usr/libexec/sendmail/sendmail > > > > -send-mail /usr/libexec/sendmail/sendmail > > > > -mailq /usr/libexec/sendmail/sendmail > > > > -makemap /usr/libexec/sendmail/makemap > > > > -newaliases /usr/libexec/sendmail/sendmail > > > > -hoststat /usr/libexec/sendmail/sendmail > > > > -purgestat /usr/libexec/sendmail/sendmail > > > > +sendmail /usr/local/libexec/sendmail/sendmail > > > > +send-mail /usr/local/libexec/sendmail/sendmail > > > > +mailq /usr/local/libexec/sendmail/sendmail > > > > +makemap /usr/local/libexec/sendmail/makemap > > > > +newaliases /usr/local/libexec/sendmail/sendmail > > > > +hoststat /usr/local/libexec/sendmail/sendmail > > > > +purgestat /usr/local/libexec/sendmail/sendmail > > > > .Ed > > > > .Sh SEE ALSO > > > > .Xr mail 1 , > > > > > >