[Pkg-mailman-hackers] Bug#504695: mailman: README.Exim4.Debian has wrong group and router settings listed

[Lionel Elie Mamane]

Hi,

Thank you for your problem report on the Debian package of Mailman.

On Thu, Nov 06, 2008 at 07:40:32PM +0900, Kris Popendorf wrote:

> The MAILMAN_GROUP line in the README.Exim4.Debian docs suggest a
> wrong group line (the files it needs to read are gid list, not
> daemon).

Actually, because of a patch in the Debian package compared to
upstream, the MAILMAN_GROUP (which originally is supposed to be the
gid under which mailman has to run, nothing to do with any file it has
to read) doesn't matter, as long as it is < 1000 (or < 500, I don't
remember exactly). But yeah, "list" is more right than daemon in
nitpicking-kind-of-way. Yes. (We already had that mentioned in another
bug somewhere...)

> Also the condition in the suggested mailman_router is (I assume)
> outdated and wrong.

No, it is not. The way your patch modifies it breaks the handling of
virtual domains, if not the whole setup. The MAILMAN_HOME/data/aliases
file contains full email addresses, not only localparts. The whole
point of that lookup is to check that this email address is in that
domain or not.

> --- old/README.Exim4.Debian	2008-11-06 19:27:56.739443959 +0900
> +++ fixed/README.Exim4.Debian	2008-11-06 19:28:42.291445556 +0900
> @@ -30,7 +30,7 @@
>  # User and group for Mailman, should match your --with-mail-gid
>  # switch to Mailman's configure script.
>  MAILMAN_USER=list
> -MAILMAN_GROUP=daemon
> +MAILMAN_GROUP=list
>  ----------------------  END EXIM4 MAIN    ----------------------------
>  
>  
> @@ -58,7 +58,7 @@
>  # Mailman lists
>  mailman_router:
>    domains = +local_domains
> -  condition = ${lookup{$local_part@$domain}lsearch{MAILMAN_HOME/data/virtual-mailman}{1}{0}}
> +  condition = ${lookup{$local_part}lsearch{MAILMAN_HOME/data/aliases}{1}{0}}
>    require_files = MAILMAN_HOME/lists/$local_part/config.pck
>    driver = accept
>    local_part_suffix_optional