[Systers-dev] Mailman bug #590155 + #495331

[erica]

Hi all,

Further research reveals that Anna made a fix that stopped a crash, but 
instead kicks out the error message that Robin was seeing per the 
initial comment in bug report 495331.  Here ( 
http://bazaar.launchpad.net/~systers-soc/systers/stable/revision/71 ) is 
the revision in the stable branch which contains both the crash fix and 
error message.

I was unable to reproduce the error message, hence the diagnostic patch 
I proposed.

Anna, thanks for the background info, it was very helpful!  Do you have 
a test case for the crash?  I think it would be the test case for bug 
495331 as well.

erica

On 6/19/2010 3:41 AM, Anna Granudd wrote:
> Hi,
> On Fri, Jun 18, 2010 at 11:52 PM, Robin Jeffries <robin at jeffries.org 
> <mailto:robin at jeffries.org>> wrote:
>
>     For the footer message, it's possible that this bug got fixed and not
>     updated.  I haven't had time to track that down yet.  Will do this
>     weekend
>     if events don't overwhelm me.
>
>
> I checked bug #495331 and compared it to the fix Robin mentioned from 
> last year (can be found here [1]) and it seems it is not the same bug 
> after all (the old fix catches the exception and enters the empty 
> string, it was a fix for a crash as a result of missing footers for 
> non dlists if I recall correctly - the bug itself seems to have been 
> deleted by Launchpad since it was fixed about a year ago), although it 
> is really strange that the 'footer-text' got lost. I guess we need to 
> find out where it should get set and why it's not any more...
>
> Anna
>
> [1] 
> http://bazaar.launchpad.net/~systers-dev/systers/wishlist/revision/76#Mailman/Handlers/Decorate.py 
> <http://bazaar.launchpad.net/%7Esysters-dev/systers/wishlist/revision/76#Mailman/Handlers/Decorate.py>
>
>
>
>