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

libmdp does not build against current stable øMQ and CZMQ #48

Open
litch opened this issue Apr 6, 2014 · 2 comments
Open

libmdp does not build against current stable øMQ and CZMQ #48

litch opened this issue Apr 6, 2014 · 2 comments

Comments

@litch
Copy link

litch commented Apr 6, 2014

I just spent a few hours trying to build this against øMQ 4.0.4 and CZMQ 2.1.0 (current stable) to no avail. I did get it to build against øMQ 3.2.4 / CZMQ 2.0.3.

The backtrace I was getting is:

gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../include  -pedantic -Werror -Wall -Wno-unused -D_GNU_SOURCE -DLINUX -D_REENTRANT -D_THREAD_SAFE   -I/usr/local/include -MT engine.o -MD -MP -MF .deps/engine.Tpo -c -o engine.o engine.c
engine.c: In function 's_order_update':
engine.c:55:5: error: too many arguments to function 'zmsg_pushstr'
/usr/local/include/zmsg.h:102:5: note: declared here
make[2]: *** [engine.o] Error 1
make[2]: Leaving directory `/majordomo/libmdp/src'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/majordomo/libmdp/src'
make: *** [all-recursive] Error 1

Build was happening on ubuntu 12.0.4.

Maybe I'll take a look at it, but I'm not too familiar with this.

@hintjens
Copy link
Member

hintjens commented Apr 6, 2014

I'll fix this... there were some CZMQ api changes, for security.

hintjens added a commit to hintjens/majordomo that referenced this issue Apr 6, 2014
@litch
Copy link
Author

litch commented Apr 6, 2014

Thank you @hintjens, I really appreciate it. I'm really impressed by iMatix and this whole system!

lerwys added a commit to lnls-dig/majordomo that referenced this issue May 2, 2014
The new czmq API (2.2.0) changed the zsocket_bind ()
signature. This, together with commit e6b3a5a
fix issue zeromq#48.
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 a pull request may close this issue.

2 participants