Commit Graph

366 Commits

Author SHA1 Message Date
Simon Giesecke
7d53a04e81 Set version to 4.3.1 to prepare bugfix release 2018-08-21 13:51:17 +02:00
Simon Giesecke
120c7ae3f2 Problem: no test case for previously existing send functionality
Solution: added test case
2018-08-21 13:50:02 +02:00
Simon Giesecke
ab09f5da98 Problem: addition of new single-argument message_t introduces ambiguity when calling e.g. socket_t::send
Solution: make single-argument constructor explicit
2018-08-21 13:50:02 +02:00
Luca Boccassi
8ff83257ac
Merge pull request #264 from sigiesec/start-4-4-0
Update version to 4.4.0 for next development iteration
2018-08-15 17:17:20 +01:00
Simon Giesecke
f5963ceade Update version to 4.4.0 for next development iteration 2018-08-15 17:55:43 +02:00
Luca Boccassi
213da0b04a
Merge pull request #256 from Tulon/radio_dish_support
Add support for RADIO/DISH sockets if draft API is enabled
2018-07-15 12:06:19 +01:00
Joseph Artsimovich
751f27d635 Add support for RADIO/DISH sockets if draft API is enabled
This commit introduces new socket_type enumeration values as well
as the following supporting functions:

socket_t::join()
socket_t::leave()
message_t::group()
message_t::set_group()
2018-07-13 16:01:15 +03:00
Luca Boccassi
73f171abb2
Merge pull request #255 from kurdybacha/issue_fix
Problem: #209 and monitor_t tests not event driven
2018-07-05 09:05:57 +01:00
Pawel Kurdybacha
57454dff4b Problem: monitor_t tests not event driven
Solution: instead of waiting for fixed amount of time for events, react
as soon as events are triggered.

- Total running time of unittest reduced 10x (from ~300ms to 30ms).
- Reduced code duplication by reusing testutil's constructs.
2018-07-05 00:23:31 +01:00
Pawel Kurdybacha
4a066be66a Problem: #209 cppzmq fails with cmake 3.6.3 and above
Solution: remove problematic cmake lines as they seem to be not necessary.
2018-07-05 00:12:34 +01:00
Simon Giesecke
837c0c9a97
Merge pull request #250 from esromneb/master
editing "typename I" to "typename T" due to error: expected nested-na…
2018-06-15 08:55:50 +02:00
Ben
a604af95d7 editing "typename I" to "typename T" due to error: expected nested-name-specifier before ‘(’ token
on c++ (Ubuntu 5.4.0-6ubuntu1~16.04.9) 5.4.0 20160609
2018-06-10 23:54:20 -07:00
Luca Boccassi
97d2cb2498
Merge pull request #249 from sigiesec/add-monitor-tests
Add first tests for monitor_t and fix monitor_t::abort
2018-06-05 16:39:58 +01:00
Simon Giesecke
58ffef7190 Problem: no tests for monitor_t::abort
Solution: add a test and fix implementation of monitor_t::abort to make it usable
2018-06-05 17:27:32 +02:00
Simon Giesecke
ec63fb3485 Problem: no test case for zmq::monitor_t::init
Solution: added test case
2018-06-05 17:27:28 +02:00
Simon Giesecke
e0b5629a28
Merge pull request #248 from kurdybacha/readme_update
Problem: README build badges too generic
2018-06-04 17:49:17 +02:00
Pawel Kurdybacha
29d3990770 Problem: README build steps can be simpler.
Solution: After latest fixes to cmake configuration it is not required
to pass include to libzmq. Please refer to demo/ for reference.
2018-06-04 16:46:41 +01:00
Pawel Kurdybacha
bfb3a5d6f7 Problem: README build badges too generic
Solution: Show Linux, OSX and Windows labels on build badges for more
visibility on supported platforms.

Additionally added license badge.
2018-06-04 16:39:38 +01:00
Constantin Rack
edc14b64fd
Merge pull request #247 from kurdybacha/coveralls
Problem: Coverage with coveralls not working
2018-06-04 07:27:29 +02:00
Pawel Kurdybacha
9a7a85703c Problem: Coverage with coveralls not working
Solution: Enable coveralls for one linux build target and fix coveralls
configuration.
2018-06-03 19:09:44 +01:00
Luca Boccassi
3c14d37d8b
Merge pull request #243 from sigiesec/fail-if-libzmq-not-found
Problem: cmake build succeeds even if libzmq not found
2018-05-25 10:25:45 +01:00
Simon Giesecke
c69c1f903f Problem: cmake build succeeds even if libzmq not found
Solution: raise a FATAL_ERROR if not found, and improve diagnostic output
2018-05-25 11:13:32 +02:00
Luca Boccassi
1f14f9a34e
Merge pull request #240 from kurdybacha/draft-git
Problem: Travis requires sudo and draft not enabled for git repo
2018-05-23 22:14:17 +01:00
Pawel Kurdybacha
115bfffc50 Problem: Travis requires sudo
sudo was required for for pkg-config libzmq build as cppzmq could not
find not installed pkg-config libzmq.

Solution: cppzmq to use cmake prefix paths for pkg config paths as well.
2018-05-23 19:18:28 +00:00
Pawel Kurdybacha
9d058b3a5e Problem: draft not explicitly enabled on Windows CI 2018-05-23 19:15:43 +01:00
Pawel Kurdybacha
380e39856b Problem: draft build not enabled for git repo
cppzmq does not follow other zeromq projects where draft builds are
enabled by default for git repository.

Solution: revert back code where ENABLE_DRAFTS in set to ON if .git
directory exists
2018-05-23 18:34:44 +01:00
Simon Giesecke
7023764834
Merge pull request #235 from Tulon/master
Add message_t::routing_id() and set_routing_id()
2018-05-20 09:25:20 +02:00
Simon Giesecke
3e56f561cc
Merge pull request #236 from kurdybacha/cmake-tweaks
Problem: cppzmq build broken with C++11 compiler and git cloned repo.
2018-05-20 09:05:33 +02:00
Pawel Kurdybacha
099bcfc4a4 Problem: cppzmq build broken with C++11 compiler and git cloned repo.
Default build, ./ci_build.sh without any arguments, which causes to run
with draft disabled, does not work properly for git cloned repository and
C++11 compiler.

Two issues:
1. For git cloned repository ENABLE_DRAFTS is ON by default but libzmq
compiled build without drafts .Travis did not catch that because default build
runs on non C++11 compiler.
2. testutil.hpp does not build because of missing draft guards.

Solution 1: Remove check for presence of .git for enabling draft API as it
is confusing to use with ENABLE_DRAFTS flag and there should be only one
explicit way to enable draft build.

Solution 2: add missing draft guards in testutil.hpp for server/client
socket in use there.

Solution 3: add extra Travis build covering C++11 compiler and non
draft enabled build.
2018-05-20 07:53:56 +01:00
Simon Giesecke
d487e67249
Merge pull request #234 from kurdybacha/cmake-tweaks
Problem: cppzmq needs to be installed for pkg-config libzmq
2018-05-19 11:30:42 +02:00
Joseph Artsimovich
d4374cbebe Add message_t::routing_id() and set_routing_id()
Setting a routing id is necessary when sending a message through a
ZMQ_SERVER socket. See [1] for more details.

[1] http://api.zeromq.org/4-2:zmq-socket#toc5
2018-05-19 08:44:41 +03:00
Pawel Kurdybacha
869ebe111a Problem: wrong version of libzmq in README.
Solution: Change it to CI supported 4.2.0
2018-05-18 22:42:13 +01:00
Pawel Kurdybacha
ecbcf10ddf Problem: cppzmq needs to be installed for pkg-config libzmq
Solution: Make sure that FinZeroMQ.cmake is present in cppzmq's build
(binary) directory
2018-05-18 22:36:35 +01:00
Simon Giesecke
b11e44550a
Merge pull request #232 from kurdybacha/pkgconfig-build
Problem: libzmq pkg-config build not covered by CI
2018-05-17 12:30:37 +02:00
Pawel Kurdybacha
265a1071e2 Problem: Default Travis build variables not defined.
Solution: simplify Travis config file by defining default build options
DRAFT and ZMQ_VERSION in ci_build.sh script
2018-05-16 21:24:38 +01:00
Pawel Kurdybacha
88bb8b1980 Problem: demo does not build for libzmq pkg-config case.
It seems to be a problem with cppzmq where FindZeroMQ.cmake is not
found without installing cppzmq.

Solution: install cppzmq for pkg-config build type.
2018-05-16 21:13:39 +01:00
Pawel Kurdybacha
9b0dd89cd9 Problem: Travis container with sudo long start
Solution: Move sudo required build to top to start sooner to get better
overall build time.
2018-05-16 21:13:39 +01:00
Pawel Kurdybacha
8dfec111f7 Problem: libzmq pkg-config build not tested
Solution: build and install libzmq using autogen tools.
2018-05-16 21:13:39 +01:00
Luca Boccassi
6fe3d54359
Merge pull request #230 from kurdybacha/osx-build
Problem: OSX build not enabled in Travis
2018-05-13 17:29:42 +01:00
Pawel Kurdybacha
4b6c4b6289 Problem: OSX not present in supported OS section.
Solution: README file updated.
2018-05-13 17:08:03 +01:00
Pawel Kurdybacha
fe56665b06 Problem: redundant addson and os section in Travis config
Solution: sections removed
2018-05-13 16:57:53 +01:00
Pawel Kurdybacha
8d84f9a1af Problem: cmake already pre-installed on OSX
Solution: do not try to install cmake on OSX and add clang as used
compiler explicitly
2018-05-13 16:57:24 +01:00
Pawel Kurdybacha
6cd57fad17 Problem: not all cores used on Travis
Solution: As Travis enables 2 cores let's use them both
2018-05-13 16:57:24 +01:00
Pawel Kurdybacha
3807107f56 Problem: coveralls does not install on OSX
Solution: enable coverall on linux only.
2018-05-13 16:57:24 +01:00
Pawel Kurdybacha
4311d1d933 Problem: enabled sudo for travis build
After previous changes `sudo` is not required anymore.
Solution: disable sudo.
2018-05-13 16:57:24 +01:00
Pawel Kurdybacha
0fd3e165f3 Problem: OSX build does not start first.
Solution: remove matrix and include build combination manually.
2018-05-13 16:05:13 +01:00
Pawel Kurdybacha
1fc0675a83 Problme: OSX build does not use clang
Default compiler on OSX is clang so it should be default.
Solution: switch from gcc to clang for OSX build.
2018-05-13 15:55:10 +01:00
Pawel Kurdybacha
738d44dc88 Problem: OSX build not enabled
Solution: Enable OSX build with DRAFT and latest libzmq
2018-05-13 15:44:46 +01:00
Pawel Kurdybacha
21350a3fd3 Problem: Only one version of libzmq tested
Solution: use libzmq 4.2.4, last two releases in use now.
2018-05-13 15:30:56 +01:00
Pawel Kurdybacha
a5578a6f25 Problem: libzmq 4.2.0 cmake build broken
It seems that there is a bug in libzmq 4.2.0 cmake configs that prevents
linking to it without installing.

Solution: disable libzmq 4.2.0 for cmake builds
2018-05-13 15:30:56 +01:00