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

add patch_pyzmq() for backporting a few changes from newer pyzmq #1652

Merged
merged 4 commits into from Apr 24, 2012

Conversation

minrk
Copy link
Member

@minrk minrk commented Apr 24, 2012

  • fixes missing constants in super-old pyzmq
  • define ioloop.install if not defined
  • never allow jsonlib to be used by zmq.utils.jsonapi

The last one should close #1520

* missing constants in pyzmq < 2.1.9
* avoid jsonlib with pyzmq < 2.2.0
ensures patch_pyzmq() has been applied before any containing files are imported.
@fperez
Copy link
Member

fperez commented Apr 24, 2012

Looks great and clean, thanks! Merging now.

fperez added a commit that referenced this pull request Apr 24, 2012
add patch_pyzmq() for backporting a few changes from newer pyzmq

* fixes missing constants in super-old pyzmq
* define ioloop.install if not defined
* never allow jsonlib to be used by zmq.utils.jsonapi (#1520).

Closes #1520.
@fperez fperez merged commit 64a5061 into ipython:master Apr 24, 2012
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
add patch_pyzmq() for backporting a few changes from newer pyzmq

* fixes missing constants in super-old pyzmq
* define ioloop.install if not defined
* never allow jsonlib to be used by zmq.utils.jsonapi (ipython#1520).

Closes ipython#1520.
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.

kernel crash when showing tooltip (?)
2 participants