On Wed, May 31, 2017 at 11:59:08AM +0100, Stuart Henderson wrote:
> On 2017/05/31 11:10, Jason McIntyre wrote:
> > On Wed, May 31, 2017 at 11:49:18AM +0200, Antoine Jacoutot wrote:
> > > On May 31, 2017 11:35:28 AM GMT+02:00, Consus <con...@gmx.com> wrote:
> > > >OpenBSD defaults to file table now so there is no need in running
> > > >newaliases(8).
> > > >---
> > > > etc/mail/aliases | 8 ++------
> > > > 1 file changed, 2 insertions(+), 6 deletions(-)
> > > >
> > > >diff --git a/etc/mail/aliases b/etc/mail/aliases
> > > >index c1ac04b5a81..045b2b2a456 100644
> > > >--- a/etc/mail/aliases
> > > >+++ b/etc/mail/aliases
> > > >@@ -1,12 +1,8 @@
> > > > #
> > > > #       $OpenBSD: aliases,v 1.64 2017/03/18 21:18:01 florian Exp $
> > > > #
> > > >-#  Aliases in this file will NOT be expanded in the header from
> > > >-#  Mail, but WILL be visible over networks or from
> > > >/usr/libexec/mail.local.
> > > >-#
> > > >-#       >>>>>>>>>>      The program "newaliases" must be run after
> > > >-#       >> NOTE >>      this file is updated for any changes to
> > > >-#       >>>>>>>>>>      show through to smtpd.
> > > >+#  Aliases in this file will NOT be expanded in the header from Mail,
> > > >but WILL
> > > >+#  be visible over networks or from /usr/libexec/mail.local.
> > > > #
> > > > 
> > > > # Basic system aliases -- these MUST be present
> > > >-- 
> > > >2.13.0
> > > 
> > > I proposed the same a while ago but people preferred to keep it in regard 
> > > to other MTA. Maybe it's time to revisit ?
> > > 
> > > 
> > 
> > well what's there now is incorrect, so i think something needs to
> > happen. even if we just prefix the text with "For databases" or
> > something.
> 
> But you need "smtpctl update table aliases" instead don't you? (At least
> that is how I read the manual).
> 
> (It would seem useful if "newaliases" did whatever is necessary for the
> table type you have in use so you don't need to think about it and the
> documentation can be simple..)
> 

It is not that simple because newaliases works when you have one aliases
database (e.g. /etc/mail/aliases). This is the case on the default setup
but smtpd supports per-rule aliases mappings and for example the MX that
I run for poolp.org and opensmtpd.org has two different sets of aliases,
one for each domain, and now newaliases can't work anymore.

We can hack it so it works in some cases but it will never be able to do
the work correctly for non-default configurations and we will still have
to provide and document the smtpctl command.

-- 
Gilles Chehade

https://www.poolp.org                                          @poolpOrg

Reply via email to