Package: release.debian.org
Severity: normal
Tags: stretch
User: release.debian....@packages.debian.org
Usertags: pu
Hi,
It was discovered that the SpamAssassin integration in Mailman (a Debian
specific piece of code) does not work in stretch. This is not enabled by
default but used often, and broke existing Mailman setups.
Attached patch rectifies the problem as confirmed by several users in
the bug reports. It is also in unstable and testing for a week without
reports of problems.
So please accept this stable update to fix this functionality.
Thanks,
Thijs
diff -Nru mailman-2.1.23/debian/changelog mailman-2.1.23/debian/changelog
--- mailman-2.1.23/debian/changelog 2016-09-13 18:01:59.000000000 +0200
+++ mailman-2.1.23/debian/changelog 2017-09-14 12:23:04.000000000 +0200
@@ -1,3 +1,10 @@
+mailman (1:2.1.23-1+deb9u1) stretch; urgency=medium
+
+ * Fixed broken dependencies in SpamAssassin.py (Closes: #838288).
+ Thanks Stephen Rothwell for the patch.
+
+ -- Thijs Kinkhorst <th...@debian.org> Thu, 14 Sep 2017 12:23:04 +0200
+
mailman (1:2.1.23-1) unstable; urgency=medium
* New upstream release.
diff -Nru mailman-2.1.23/debian/contrib/SpamAssassin.py
mailman-2.1.23/debian/contrib/SpamAssassin.py
--- mailman-2.1.23/debian/contrib/SpamAssassin.py 2016-09-13
18:01:59.000000000 +0200
+++ mailman-2.1.23/debian/contrib/SpamAssassin.py 2017-09-14
09:49:35.000000000 +0200
@@ -28,7 +28,6 @@
from Mailman import Errors
from Mailman.Logging.Syslog import syslog
from Mailman.Handlers import Hold
-from Mailman.Handlers.Moderate import matches_p
SPAMD_HOST = getattr(mm_cfg, 'SPAMASSASSIN_HOST', '')
DISCARD_SCORE = getattr(mm_cfg, 'SPAMASSASSIN_DISCARD_SCORE', 10)
@@ -78,7 +77,7 @@
if MEMBER_BONUS != 0:
for sender in msg.get_senders():
if mlist.isMember(sender) or \
- matches_p(sender, mlist.accept_these_nonmembers,
mlist.internal_name()):
+ mlist.GetPattern(sender, mlist.accept_these_nonmembers,
at_list='accept_these_nonmembers'):
score -= MEMBER_BONUS
break
