Bug 220833 - mail/sympa: provide correct path to postalias and make the sympa_newaliases.pl work with postfix
Summary: mail/sympa: provide correct path to postalias and make the sympa_newaliases.p...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mathieu Arnold
URL:
Keywords: easy
Depends on:
Blocks:
 
Reported: 2017-07-18 14:10 UTC by Mathieu Arnold
Modified: 2017-07-18 18:56 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (dgeo)
koobs: merge-quarterly?


Attachments
v1 (853 bytes, patch)
2017-07-18 14:10 UTC, Mathieu Arnold
no flags Details | Diff
Same patch + pkg-message updated (1.74 KB, patch)
2017-07-18 17:38 UTC, geoffroy desvernay
dgeo: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mathieu Arnold freebsd_committer freebsd_triage 2017-07-18 14:10:04 UTC
Created attachment 184471 [details]
v1

Without this, it burps out a:

# sudo -u sympa /usr/local/libexec/sympa/sympa_newaliases-wrapper
Can't exec "/usr/sbin/postalias": No such file or directory at /usr/local/libexec/sympa/sympa_newaliases.pl line xxx.

And we can't use --with-postalias, because it does not exist at build time.
Comment 1 geoffroy desvernay 2017-07-18 17:38:06 UTC
Created attachment 184480 [details]
Same patch + pkg-message updated

Sorry I didn't caught this one before (here we use postfix but with regexp maps)

If you dare I'll add here a little change in pkg-message (unrelated, but doesn't worth it's PR).

Thank you !
Comment 2 geoffroy desvernay 2017-07-18 17:40:20 UTC
I can't set values here, but I'd vote to merge it to quarterly, this can't hurt anyone and can help some !
Comment 3 Mathieu Arnold freebsd_committer freebsd_triage 2017-07-18 17:52:31 UTC
Mmm, I use a pcre for the virtual, but not the aliases, could you share your aliases file ? 0:-)
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-07-18 17:52:50 UTC
A commit references this bug:

Author: mat
Date: Tue Jul 18 17:52:18 UTC 2017
New revision: 446152
URL: https://svnweb.freebsd.org/changeset/ports/446152

Log:
  Fix usage with postfix's postalias, also update pkg-message.

  PR:		220833
  Submitted by:	mat and maintainer
  MFH:		2017Q3
  Sponsored by:	Absolight

Changes:
  head/mail/sympa/Makefile
  head/mail/sympa/files/pkg-message.in
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-07-18 17:53:53 UTC
A commit references this bug:

Author: mat
Date: Tue Jul 18 17:53:39 UTC 2017
New revision: 446153
URL: https://svnweb.freebsd.org/changeset/ports/446153

Log:
  MFH: r446152

  Fix usage with postfix's postalias, also update pkg-message.

  PR:		220833
  Submitted by:	mat and maintainer
  Sponsored by:	Absolight

Changes:
_U  branches/2017Q3/
  branches/2017Q3/mail/sympa/Makefile
  branches/2017Q3/mail/sympa/files/pkg-message.in
Comment 6 geoffroy desvernay 2017-07-18 18:56:43 UTC
@mat I (once more) wrote too quickly - I'm not having the problem anyhow :-/

I just configured it in sympa.conf
sendmail_aliases        /sympa/mail/sympa_aliases
aliases_db_type hash
aliases_program         /usr/local/sbin/postalias

Working here, last list was added yesterday:
# ls -al /sympa/mail 
total 283
drwxr-xr-x  2 sympa  postfix        4 May  2 19:06 .
drwxr-xr-x  8 root   wheel          8 Oct 27  2012 ..
-rw-r--r--  1 sympa  sympa     943144 Jul 17 16:16 sympa_aliases
-rw-r--r--  1 sympa  postfix  1343488 Jul 17 16:16 sympa_aliases.db

I think I'm falling here (from /usr/local/libexec/sympa/sympa_newaliases.pl):
114 if ($aliases_program =~ m{\A/}) {
115     $log->syslog('debug2', 'Executing "%s %s"',
116         $aliases_program, $aliases_file);
117 
118     exec $aliases_program, $aliases_file;
119 }

I think that's why I didn't got this bug… Thank you !