Mark Sapiro wrote: > Tokio Kikuchi wrote: > >>We may have to patch against this email package parsedate bug. >>I've just uploaded a patch on SF tracker. Please someone review this >>before I commit in the CVS (this weekend, maybe). > > > I have looked at the patch in the tracker. > > Caveat: I haven't tested anything - this is just based on my reading. > > I think the patch is good. The issue I see is that Scrubber.py may not > currently be doing the right thing if parsedate() returns None. > > Consider the attached patch for Scrubber.py in addition to the patch in > the tracker.
Well, the logic may be unclear but calculate_attachment_dir() tries again to guess the real date of message arrival because it may be called from bin/arch. I think the code should be cleaned up but since we are now dealing with the email parsedate bug and it should be safe to limit our patch to this purpose. -- Tokio Kikuchi, tkikuchi@ is.kochi-u.ac.jp http://weather.is.kochi-u.ac.jp/ _______________________________________________ Mailman-Developers mailing list [email protected] http://mail.python.org/mailman/listinfo/mailman-developers Mailman FAQ: http://www.python.org/cgi-bin/faqw-mm.py Searchable Archives: http://www.mail-archive.com/mailman-users%40python.org/ Unsubscribe: http://mail.python.org/mailman/options/mailman-developers/archive%40jab.org Security Policy: http://www.python.org/cgi-bin/faqw-mm.py?req=show&file=faq01.027.htp
