Mailing List Archive: Python: Bugs

[issue756982] mailbox should use email not rfc822

Index | Next | Previous | View Threaded

report at bugs

May 10, 2008, 2:14 PM

Post #1 of 10 (464 views)
Permalink

[issue756982] mailbox should use email not rfc822

Humberto Diogenes <humberto [at] digi> added the comment:

synx's patch wasn't applying cleanly and broke 2 maildir tests. I'm
posting one with updated tests and documentation. Still need to get rid of
rfc822 in test_mailbox, though.

Question: mailbox.py has one section marked as "classes from the original
module (for backward compatibility)". Shouldn't these be removed in py3k?

----------
nosy: +hdiogenes
Added file: http://bugs.python.org/file10272/mailbox.py.patch2

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 10, 2008, 3:19 PM

Post #2 of 10 (456 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

A.M. Kuchling <lists [at] amk> added the comment:

I think removing those old classes in 3.0 would be very sensible.

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 10, 2008, 8:07 PM

Post #3 of 10 (452 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Humberto Diogenes <humberto [at] digi> added the comment:

I created issue 2814 with a patch to remove those old classes.

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 12:33 AM

Post #4 of 10 (446 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Changes by Humberto Diogenes <humberto [at] digi>:

----------
keywords: +patch
Added file: http://bugs.python.org/file10279/mailbox-replace_rfc822.patch

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 12:33 AM

Post #5 of 10 (445 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Changes by Humberto Diogenes <humberto [at] digi>:

Removed file: http://bugs.python.org/file10272/mailbox.py.patch2

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 2:02 AM

Post #6 of 10 (446 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Georg Brandl <georg [at] python> added the comment:

I removed the old classes in 3k. There remains one usage of rfc822, as
the default "factory" of Maildir. It's replaced by None in the patch
here; is that safe to do?

----------
nosy: +georg.brandl

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 12:41 PM

Post #7 of 10 (435 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Humberto Diogenes <humberto [at] digi> added the comment:

> rfc822 is replaced by None in the patch here; is that safe to do?

Yes. That's what mailbox documentation says:

Parameter factory is a callable object that accepts a file-like message
representation [...] and returns a custom representation. If factory is
None, MaildirMessage is used as the default message representation.

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 1:51 PM

Post #8 of 10 (434 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Georg Brandl <georg [at] python> added the comment:

Removed the last rfc822 reference from 3k's mailbox.py in r63091.

----------
resolution: -> fixed
status: open -> closed

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 2:22 PM

Post #9 of 10 (436 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Humberto Diogenes <humberto [at] digi> added the comment:

Georg, any special reason for not removing rfc822 references from
test_mailbox? That section of the patch was not merged.

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

report at bugs

May 11, 2008, 2:31 PM

Post #10 of 10 (436 views)
Permalink

[issue756982] mailbox should use email not rfc822 [In reply to]

Georg Brandl <georg [at] python> added the comment:

Okay, removed that too in r63096.

____________________________________
Tracker <report [at] bugs>
<http://bugs.python.org/issue756982>
____________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/list-python-bugs%40lists.gossamer-threads.com

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads