[Mailman-Developers] Problem in 2.1b1's contrib/qmail-to-mailman.py

Barry A. Warsaw barry@zope.com
Fri, 5 Apr 2002 13:53:26 -0500


>>>>> "BW" == Bill Wagner <bwagner@potpie.org> writes:

    BW> First off, I'm sorry if this isn't the appropriate forum, but
    BW> I felt it more appropriate than mailman-users.

This one's fine!

    BW> I'm running a Mandrake 8.0 box with qmail and have 2.0.9
    BW> running fine but am testing 2.1b1 and have run into a problem.

    BW> When passing mail through qmail-to-mailman.py, lines 87 and 88
    BW> point to $prefix/mail/wrapper.  I've done plenty of test
    BW> installs of 2.1b1, but in none ofthem is there ever a
    BW> $prefix/mail/wrapper.  There's a mail/mailman, but that'snot
    BW> what qmail-to-mailman.py looks for.  I did a test and modified
    BW> it to point tomail/mailman, but that results in a deferral
    BW> like:

I don't use qmail so it's up to the community to keep the qmail
documentation and contribs up-to-date.  

    BW> So I was wondering if this is a known issue and if anyone
    BW> might have an idea of a quick workaround.  Is mail/wrapper now
    BW> obsolete?

Yes, mail/wrapper is obsolete; it's been renamed mail/mailman to avoid
unnecessary conflicts with MD's wrapper program.  A dumb replace
yields the following patch (committed to cvs, though if the whole file
needs updating, please send contributions).

-Barry

Index: qmail-to-mailman.py
===================================================================
RCS file: /cvsroot/mailman/mailman/contrib/qmail-to-mailman.py,v
retrieving revision 2.1
retrieving revision 2.2
diff -u -r2.1 -r2.2
--- qmail-to-mailman.py	7 Sep 2001 23:18:47 -0000	2.1
+++ qmail-to-mailman.py	5 Apr 2002 18:53:01 -0000	2.2
@@ -84,8 +84,8 @@
             local = re.sub(i[0],"",local)
 
     if os.path.exists(local):
-        os.execv(MailmanHome + "/mail/wrapper",
-                 (MailmanHome + "/mail/wrapper", type, local))
+        os.execv(MailmanHome + "/mail/mailman",
+                 (MailmanHome + "/mail/mailman", type, local))
     else:
         bounce()
     sys.exit(111)