[Mailman-Developers] It happened again!

[Dan Mick]

> > Dammit, last night I upgraded to latest CVS, and all my nomails got
> > reset again!
> > 
> > Argh.
> 
> ...and it's this line, right, here, right Barry?
> 
>     # 2.1alpha3 -> 2.1alpha4.  The DisableDelivery flag is now moved into
>     # get/setDeilveryStatus().  This must be done after the addresses are
>     # canonicalized.
>     l.delivery_status = {}		<<<<<<<<<<
>     for k, v in l.user_options.items():
>         if l.getMemberOption(k, mm_cfg.DisableDelivery):
>             # Convert this flag into a legacy disable
>             l.setDeliveryStatus(k, UNKNOWN)
>             l.setMemberOption(k, mm_cfg.DisableDelivery, 0)

Perhaps this is right:


Index: versions.py
===================================================================
RCS file: /cvsroot/mailman/mailman/Mailman/versions.py,v
retrieving revision 2.17
diff -r2.17 versions.py
338c338,339
<     l.delivery_status = {}
---
>     if not hasattr(l, 'delivery_status')
>         l.delivery_status = {}