snakebite icon indicating copy to clipboard operation
snakebite copied to clipboard

Improve python3 support

Open tobes opened this issue 8 years ago • 2 comments

In python 3 long and int types have been unified and it is no longer legal to use the L postfix to declare a long. As this postfix is used in snakebite it fails under python 3. This commit improves python 3 support for some use cases.

It is not intended to make snakebite fully python 3 compatible in any way but just to remove a common issue. This changes allows limited functionality under python 3 which is enough for many use-cases.


This is basically a clean version of #203 and a simplification of #239 although maybe the changes to channel.py should be considered, but since they do not affect me I've not added them.

The impact on the codebase is minimal and this simple change can make a huge difference to users of python 3.x

tobes avatar Jan 29 '18 17:01 tobes

I have added a second commit so that the tests pass. travis seems to have removed their python 2.6 environments. I can remove this is desired

tobes avatar Jan 29 '18 17:01 tobes

Very appreciated if the change in snakebite/client.py could be merged.

kevinneville avatar Apr 08 '20 19:04 kevinneville