Python 2 support about httptools HOT 5 CLOSED

OK, you can make a PR.

the all in the init.py fails, because I think all in Cython is a unicode object, something Python 2 doesn't like. I'd advocate changing the init.py to explicitly import everything needed instead of doing a from .. import *. This way you are in full control of what's exposed in the public API.

Good.

assertRaisesRegex which is missing. It'd need to be backported, or the test rewritten
as the regexes used aren't complicated.

Ideally, I'd like to see assertRaisesRegex ported. I'll be doing more development in httptools later (proper http protocol implementation etc), and I'm used to assertRaisesRegex.

subTest, either backported or the test rewritten.

We can write a rudimentary version for py2.

I recommend a tool like tox so we can easily run the tests on both Pythons.

OK.

the Makefile is geared towards Python 3, but doesn't do all that much. I just called the build_ext command myself with Python 2.

I'd prefer to keep the Makefile as is. Or, you can add a few more targets with a '2' suffix.

from httptools.

BTW, there is this package: https://github.com/benoitc/http-parser/
It appears it uses the same C parser and supports Python 2.

from httptools.

@faassen Are you working on any PR? Because if not (of if you haven't got too far) I'd like to close this issue. Since there is another similar package for Python 2, I see no point in migrating this one.

from httptools.

assertRaisesRegex is provided by unittest2, and is only needed for Python <= 2.6
Python 2.7 already supports it: https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertRaisesRegexp
It was renamed in Py3.2: https://docs.python.org/3.5/library/unittest.html#unittest.TestCase.assertRaisesRegex
Adding this to the top of the test (or py2 compatibility shim):

try:
    unittest.TestCase.assertRaisesRegex
except AttributeError:
    unittest.TestCase.assertRaisesRegex = unittest.TestCase.assertRaisesRegexp

Will make it just work.

from httptools.

Alright, I'm closing this PR.

from httptools.