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.

Sign up for GitHub

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

Jump to bottom

Migrate to ZMQ.pm #24

Open
2 of 3 tasks
pjf opened this issue Feb 10, 2014 · 3 comments
Open
2 of 3 tasks

Migrate to ZMQ.pm #24

pjf opened this issue Feb 10, 2014 · 3 comments
Labels
internal API wishlist ★☆☆

Comments

@pjf
Copy link
Owner

pjf commented Feb 10, 2014 •
edited
Loading

The ZMQ module for Perl supports both ZMQ2 and ZMQ3. While there's a little work that's needed to do version-sniffing for things like send() and recv(), most of it's generic. It should be pretty easy for what we're doing to use it.

  • Switch from ZMQ2 to ZMQ
  • Test under ZMQ2
  • Test under ZMQ3

Word on the street is the ZMQx::Class will probably end up being the way to go eventually, as it should support both ZMQ3 and the freshly released ZMQ4, but that can be done later.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@pjf
Copy link
Owner Author

pjf commented Feb 10, 2014

(Originally brought up in #22 )

pjf added a commit that referenced this issue Feb 10, 2014
@pjf
Switch from using ZMQ::LibZMQ2 to ZMQ for future compat.
6c3edd7 
This has not yet been tested using ZMQ3, but it *should* be
pretty close to working, as we're using the ZMQ abstractions
where possible.

Part of GH #24
@pjf
Copy link
Owner Author

pjf commented Feb 10, 2014

Leaving this open for now, because while travis is testing on ZMQ3, I don't think we're pushing packets yet.

@pjf pjf mentioned this issue Feb 10, 2014
Open
@techman83
Copy link

#52 Covers off a lot of discussion about ZMQ3/4. I read through ZMQ.pm at the time and it had taken into consideration the differences (I think the big one is multipart packets from memory). Either way, with #53 I can reliably push packets across both version 3 and 4. So in theory it "Just Works".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internal API wishlist ★☆☆
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pjf @techman83