Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ctype int for _INT_PACKER and cleanup travisci config #77

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

sarnold
Copy link

@sarnold sarnold commented Nov 25, 2019

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
Copy link
Author

sarnold commented Nov 25, 2019

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).

sarnold and others added 20 commits January 13, 2020 13:59
The error message was showing the return code instead of the length of the returned object.
Signed-off-by: Stephen L Arnold <[email protected]>
Signed-off-by: Stephen L Arnold <[email protected]>
Signed-off-by: Stephen L Arnold <[email protected]>
migrate CI to github actions
New ci using github actions
replace setuptools_scm with legacy version.py, adjust project files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants