Comment 2 for bug 411192

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

I hate to patch 'contrib' modules that I don't use as I don't know whether the resultant module does what it's supposed to, and it's really the responsibility of the contributor.

However, that said, the flaw you point out is obvious enough that I will do it in this case. Note however the following deficiencies in your patch:

1) The paths module has no var_prefix attribute. This needs to be mm_cfg.VAR_PREFIX.

2) You missed line 90

    file = paths.prefix + '/data/last_mailman_version'

needs to be changed also.