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

FileMQ does not build under OS X #72

Open
msteinhoff opened this issue Jan 28, 2016 · 6 comments
Open

FileMQ does not build under OS X #72

msteinhoff opened this issue Jan 28, 2016 · 6 comments

Comments

@msteinhoff
Copy link
Member

I tried to build filemq and go this error:

  CC       src/src_libfilemq_la-fmq_msg.lo
  CC       src/src_libfilemq_la-fmq_server.lo
In file included from src/fmq_server.c:23:
In file included from ./src/filemq_classes.h:25:
In file included from ./src/../include/filemq.h:18:
./include/filemq_library.h:53:9: error: 'WITH_DRAFTS' macro redefined [-Werror]
#define WITH_DRAFTS
        ^
./src/platform.h:185:9: note: previous definition is here
#define WITH_DRAFTS 1
        ^
1 error generated.
make[1]: *** [src/src_libfilemq_la-fmq_server.lo] Error 1
make: *** [check-recursive] Error 1

I don't have C skills and haven't the slightest clue what this means but I'd be happy to provide more info if required.

@keent
Copy link
Member

keent commented Jan 28, 2016

We'll fix this later on using zproject - for now you can comment out the line in filemq_library.h to just add double slash e.g.
//#define WITH_DRAFTS

@msteinhoff
Copy link
Member Author

Okay, now I got a bit farther, but the selftest fails:

/bin/sh ./libtool --mode=execute ./src/filemq_selftest
Running filemq selftests...
Assertion failed: (zhash_size (options) == 2), function fmq_msg_test, file src/fmq_msg.c, line 1130.
 * fmq_msg:make[2]: *** [check-local] Abort trap: 6
make[1]: *** [check-am] Error 2
make: *** [check-recursive] Error 1

Is this also something I can ignore?

@keent
Copy link
Member

keent commented Jan 28, 2016

I'll create a pull request later. For now it's creating just one option so line
1130 and 1137 should be equated to 1, not 2. ie

assert (zhash_size (options) == 1);
assert (zhash_size (cache) == 1);

@msteinhoff
Copy link
Member Author

Okay, I found another problem in line 1193 which should be:

assert (zhash_size (headers) == 1);

It now builds :)

@keent
Copy link
Member

keent commented Jan 28, 2016

Nice. It would be best if you can create a pull request since you already have the modified file :)

msteinhoff added a commit to msteinhoff/filemq that referenced this issue Jan 28, 2016
Solution: Apply changes to filemq_library.h and tests in fmq_msg.c as
suggested in issue zeromq#72.
@msteinhoff
Copy link
Member Author

Done :)

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

No branches or pull requests

2 participants