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

Pinning pyzmq requirement to 2.2.0.1 #57

Closed
wants to merge 1 commit into from

Conversation

kencochrane
Copy link

Pinning pyzmq requirement to 2.2.0.1, instead of saying anything above 2.2.0.1 is good, because it is breaking due to pyzmq 13.0
We should pin to 2.2.0.1 until we know it works fine on a higher version.

Pinning pyzmq requirement to 2.2.0.1, instead of saying anything above 2.2.0.1 is good, because it is breaking due to pyzmq 13.0
We should pin to 2.2.0.1 until we know it works fine on a higher version.
@bombela
Copy link
Member

bombela commented Feb 26, 2013

I am concern that it might bother people who also use the latest version of ZMQ directly. Hopefully tonight (PST) I will either fix the problem with the latest version of pyzmq, or accept your patch, either way, I will update the package on pypi.

@kencochrane
Copy link
Author

Any update on this? If you don't have time to pull/merge and push to pypi, just let me know and I can do it.

@bombela
Copy link
Member

bombela commented Mar 6, 2013

god dammit, totally lost track of time grrr. I am setting up a reminder to
take care of it tonight. I swear!

On Wed, Mar 6, 2013 at 2:17 PM, Ken Cochrane notifications@github.comwrote:

Any update on this?


Reply to this email directly or view it on GitHubhttps://github.com//pull/57#issuecomment-14529914
.

@kencochrane
Copy link
Author

OK, thanks.

@bombela
Copy link
Member

bombela commented Mar 7, 2013

See this commit: cf3741c

I tested it with pyzmq 13.0.0 and the older pyzmq-static 2.1.11.2, it seems to work for me

Please confirm that this is working for you with the latest master. I will then bump the version and update the package on pipy.

Best,
fx

@bombela
Copy link
Member

bombela commented Jun 10, 2013

Seems to be working with the latest version of zmq and zerorpc 0,4,2.

@bombela bombela closed this Jun 10, 2013
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.

None yet

2 participants