Yes, the comment should go as it is superfluous. Let me know if you
decide you want the BUGS section after all and I will re-do the patch.
On Sun, Sep 16, 2018 at 5:44 PM Jason McIntyre <j...@kerhand.co.uk> wrote:
>
> 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 ,
> > > >
> > >
> >
>
