re-ran c2nim on 4.2.0 and resolved diff with 4.0.x

nim-lang / nim-zmq

Nim ZMQ wrapper

https://nim-lang.github.io/nim-zmq/zmq.html

MIT License

67 stars 17 forks source link

re-ran c2nim on 4.2.0 and resolved diff with 4.0.x #4

Closed douglas-larocca closed 7 years ago

douglas-larocca commented 9 years ago

I started seeing

Assertion failed: check () (src/msg.cpp:338)
Traceback (most recent call last)
...
zmq.nim(561)    receive
SIGABRT: Abnormal termination.

This was due to a change in the message size in libzmq-4.2.0. The client sends a 32 byte message to the server which expects 64 bytes and the assertion fails. https://github.com/zeromq/libzmq/pull/1298 So it's not a bug, just the server using the wrong library version. But here's a PR that starts working on 4.2.x.

Araq commented 9 years ago

Well so if the minimum ZMQ requirement is now 4.2 this needs to be documented at least. Preferable it would check against a DLL imported version so it fails with a clear error message rather than crashing unreliably.

douglas-larocca commented 9 years ago

Thanks Araq. I'm working on support for libzmq >= 4.x. I think requiring 4.2 would probably turn most people away. For reference, right now 4.1 is in debian experimental and 4.0 is in jesse & sid.