tonysimpson / nanomsg-python

nanomsg wrapper for python with multiple backends (CPython and ctypes) should support 2/3 and Pypy
MIT License
382 stars 85 forks source link

Fix ctype int for _INT_PACKER and cleanup travisci config #77

Open sarnold opened 4 years ago

sarnold commented 4 years ago

Hi, I guess the nanomsg nng/legacy thing is still an issue (depending on individual needs) so this binding is definitely not obsolete, eg, nanoservice uses this binding and I need to use nanoservice, so I ended up having to fix the failing tests, which turned out to be a small bug in the size of the integer passed to get_int interface. This is basically that fix plus minimal .travis.yml fixes to make it build again in the default travis xenial env.

sarnold commented 4 years ago

Sorry, I didn't see the other PR #69 so I'm closing this one. Travis builds work fine with minor updates, and I noticed above your appveyor builds all work except 2 really old versions (I would just drop those unless your have a hard requirement to support 2008).