[issue18891] Master patch for content manager addtion to email package.

2013-10-16 Thread Roundup Robot
Roundup Robot added the comment: New changeset 6d12285e250b by R David Murray in branch 'default': #18891: Complete new provisional email API. http://hg.python.org/cpython/rev/6d12285e250b -- nosy: +python-dev ___ Python tracker

[issue18891] Master patch for content manager addtion to email package.

2013-10-16 Thread Eric Snow
Eric Snow added the comment: nice! Thanks for doing this. -- nosy: +eric.snow ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue18891 ___ ___

[issue18891] Master patch for content manager addtion to email package.

2013-10-12 Thread R. David Murray
R. David Murray added the comment: Updated patch to address Serhiy's review comments. Also noticed a bug and fixed it, adding a new 'is_attachment' attribute to EmailMessage/MIMEPart to do so. -- Added file: http://bugs.python.org/file32069/master_content_manager.patch

[issue18891] Master patch for content manager addtion to email package.

2013-10-11 Thread R. David Murray
R. David Murray added the comment: Final doc edits done. I will commit this next week if there are no objections, in order to get it in to alpha 4. -- Added file: http://bugs.python.org/file32055/master_content_manager.patch ___ Python tracker

[issue18891] Master patch for content manager addtion to email package.

2013-10-09 Thread R. David Murray
R. David Murray added the comment: Updating the patch to address Stephen's review comments (thanks, Stephen!). The biggest change is adding a MIMEPart class, and renaming MIMEMessage to EmailMessage. Other significant changes are: only moving 'Content-*' headers in 'make' methods, adding

[issue18891] Master patch for content manager addtion to email package.

2013-09-03 Thread Stephen J. Turnbull
Stephen J. Turnbull added the comment: I'm thinking this may be overengineering, but I may as well post it and find out for sure. :-) Is it worth encapsulating MIME types? They're really pairs as far as mail handling applications are concerned, but they have a string representation. So

[issue18891] Master patch for content manager addtion to email package.

2013-09-03 Thread R. David Murray
R. David Murray added the comment: It's an interesting thought. It bothered me to be handling them as pure strings when writing the code. It just felt wrong somehow :) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue18891

[issue18891] Master patch for content manager addtion to email package.

2013-09-03 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: On Sep 03, 2013, at 07:37 AM, Stephen J. Turnbull wrote: I'm thinking this may be overengineering, but I may as well post it and find out for sure. :-) Is it worth encapsulating MIME types? They're really pairs as far as mail handling applications are

[issue18891] Master patch for content manager addtion to email package.

2013-08-31 Thread Serhiy Storchaka
Changes by Serhiy Storchaka storch...@gmail.com: -- nosy: +serhiy.storchaka ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue18891 ___ ___

[issue18891] Master patch for content manager addtion to email package.

2013-08-30 Thread R. David Murray
New submission from R. David Murray: Since the only way to get a rietveld review link for all the changes is to post a single complete patch, I'm doing so in this issue. Feel free to review based on either this or the separate patches posted in issue 18785, issue 18860, and issue 18890.