search

zeromq / majordomo

Majordomo Project
Mozilla Public License 2.0
114 stars 52 forks source link

mdp_broker enhancements #43

Closed tclarke closed 11 years ago

tclarke commented 11 years ago

Added a daemonize option to mdp_broker. Added the ability to specify a 0mq URL for mdp_broker on the command line.

hintjens commented 11 years ago

Note that the build breaks (strict compiler):

mdp_broker.c:558:15: error: ignoring return value of ‘daemon’, declared with attribute warn_unused_result [-Werror=unused-result]

hintjens commented 11 years ago

I've a Travis CI project here: https://travis-ci.org/zeromq/majordomo

JensRantil commented 11 years ago

@hintjens Didn't Travis set a bad build status for this pull request?

hintjens commented 11 years ago

Yes, but I tend to merge without waiting for that. If the breakage annoys someone, they'll fix it, and if that doesn't happen, I'll revert it completely.

JensRantil commented 11 years ago

I see, I was just curious. No worries!