SpamAssassin documentation missing step from patch file

Bug #1666390 reported by Jeremy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Fix Released
Medium
Mark Sapiro

Bug Description

This page gives a few methods for using SpamAssassin with mailman: https://wiki.list.org/DOC/4.23%20How%20do%20I%20use%20SpamAssassin%20with%20Mailman%3F
The second methods links to a set of instructions (http://www.jamesh.id.au/articles/mailman-spamassassin/) and then gives some additional steps to take.

There are two different patch files given that you need apply to SpamAssassin.py to integrate SpamAssassin with mailman, depending on what version. The second one (just after "And beginning with Mailman 2.1.21, the following patch is required instead of the above.") isn't complete, there is an old import that needs to be deleted that isn't mentioned.

Specifically, line 31, `from Mailman.Handlers.Moderate import matches_p` needs to be deleted.

I just went through the instructions and it took a bit of debugging to find where the error logs were, and what the problem was.

Revision history for this message
Mark Sapiro (msapiro) wrote :

Thanks for the report. I updated the wiki.

Changed in mailman:
assignee: nobody → Mark Sapiro (msapiro)
importance: Undecided → Medium
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.