Message5300
The roundup.anypy.email_ module is providing backward compatibility for
versions of Python older than v2.4. Roundup no longer provides support
for anything before Python v2.5, so there is potential for removing the
module.
The following parts of roundup.anypy.email_ need to be reworked/migrated
to enable removing the module:
* FeedParser
This should be fairly straight-forward to cleanup/remove
* "continuation_ws monkey patch"
The monkey patch fixes incompatibilities between Python v2.[56] and v2.7
when generating headers. Ensuring this "magic" works relies on the
module being imported but not necessarily used (ie. in
'roundup/cgi/client.py' and 'roundup/mailer.py'). This behaviour is
non-obvious and may lead to these imports being removed because they
appear to be redundant. Implementing work-arounds in the relevant parts
of the Roundup codebase may be a better long-term solution and allow the
removal of this monkey patch.
* decode_header()
This appears to be a backport of some fixes to
email.header.decode_header() that were applied to Python 2.7/3.3+ to
provide compatibility for Python 2.[56]. Running flake8 over the code
seems to indicate that the code itself has some issues. The
decode_header() function is also used in sections of code
(roundup/mailgw.py) that are using the mimetools module, which has been
deprecated since Python v2.3. So this may required a more general cleanup. |
|
Date |
User |
Action |
Args |
2015-03-20 14:04:15 | jerrykan | set | recipients:
+ jerrykan |
2015-03-20 14:04:15 | jerrykan | set | messageid: <1426860255.33.0.770232221881.issue2550879@psf.upfronthosting.co.za> |
2015-03-20 14:04:15 | jerrykan | link | issue2550879 messages |
2015-03-20 14:04:14 | jerrykan | create | |
|