postfix-to-mailman.py does not recognize lists that end in a control extension

[Helge Willum Thingvad]

Ubuntu 10.04.4
Version: 1:2.1.13-1ubuntu0.2

The postfix-to-mailman.py script does not recognize list names like "mylist-admin" or "mylist-request".

Currently it strips off the control extension (e.g. "-admin", "-owner", etc.) from mlist before checking if it actually exists.

This patch is very simple:
- Before checking for control extension, check if list_exists(mlist)
- Import list_exists from Mailman.Utils a little earlier.

Before applying this patch, trying to post to "mylist-admin" would result in bounce message: "user unknown"

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "Check if mlist exists before stripping control extension" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

[Robie Basak (racb)]

Thank you for taking the time to report this bug and helping to make Ubuntu better.

The postfix-to-mailman.py script appears to come from packaging, rather than the upstream Mailman project. The packaging for mailman in Ubuntu is in sync with Debian at the moment, so this patch would best be forwarded there.

I also note that your patch appears to be against an old version of this script, and your patch doesn't currently apply to the current version. So some modification may be necessary.

Please could you update your patch and forward it to the Debian bug tracking system?

[Helge Willum Thingvad (helgesdk)]

The patch applies cleanly on the latest mailman for Lucid, mailman_2.1.13-1ubuntu0.2.
Yes, I know... I am running an old distro.

It seems this has been fixed in Debian Wheezy, currently mailman_2.1.15.
Same applies for the latest package in Precise.

I have attached a diff between the current Lucid and Precise versions.
As you can see, not much is different from the patch I made earlier.

Since the changes are minimal, couldn't we just copy the script from Precise and put it in an updated package for Lucid?

[Thijs Kinkhorst (kink)]

Has been fxed in 2.115-1 and up