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 ,
> > > 
> > 
> 

Reply via email to