name unicode not defined

AntonKueltz / fastecdsa

Python library for fast elliptic curve crypto

https://pypi.python.org/pypi/fastecdsa

The Unlicense

264 stars 77 forks source link

name unicode not defined #36

Closed oditorium closed 5 years ago

oditorium commented 5 years ago

Screen Shot 2019-04-27 at 17 25 50

AntonKueltz commented 5 years ago

One of the joys of writing a package that works for python2 and python3 is handling the different string / binary types... fixed in 41a9c85e77dd59cbf0a22dee252e10ba55ecfe25 and released in 6e4648105602add712c3cd3682835de3ffda7c75.

oditorium commented 5 years ago

yeah; just drop P2 support and/or have people pay you for it; at this stage everyone using P2 does it because they can't be bothered to port their legacy code base to P3, so they might as well pay for the privilege of having up to date libraries...

anyway, thanks for fixing it, and thanks for writing the library in the first place

AntonKueltz commented 5 years ago

I'll support python2 until it's EOL, only a couple more months, after that releases will be only python3 compatible.