Roundup Tracker - Issues

Message6214

Author joseph_myers
Recipients joseph_myers
Date 2018-08-20.00:42:17
Message-id <1534725739.64.0.56676864532.issue2550994@psf.upfronthosting.co.za>
In-reply-to
There is a backport of the Python 3 configparser module to Python 2:
https://pypi.org/project/configparser/

If you have this installed, it breaks running Roundup with Python 2
(example symptom: attempts to load static files with @@file URIs produce
a 404 error - I noticed this after an upgrade to Ubuntu 18.04, as on
Ubuntu 18.04 that backport is a dependency of the Python 2 version of
pylint; it also causes at least one test_cgi test failure).  The Roundup
code tries to import configparser and only imports ConfigParser if that
fails, which is normally the preferred way of handling such differences
between Python 2 and Python 3.  But this backport produces Python 2
unicode objects in cases where Roundup does not expect them (in the
example above: for the static files directory, where serve_static_file
checks for type str).

Note: this issue exists in 1.6.0 but not older releases (the
configparser handling was inserted by ESR's Python 3 fixes last year,
not those postdating the 1.6.0 release).

I'm testing a patch to use a version check for whether to use
configparser or ConfigParser and will commit that if it passes testing.
History
Date User Action Args
2018-08-20 00:42:19joseph_myerssetrecipients: + joseph_myers
2018-08-20 00:42:19joseph_myerssetmessageid: <1534725739.64.0.56676864532.issue2550994@psf.upfronthosting.co.za>
2018-08-20 00:42:19joseph_myerslinkissue2550994 messages
2018-08-20 00:42:17joseph_myerscreate