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

Problem: libzmq pkg-config build not covered by CI #232

Merged
merged 4 commits into from
May 17, 2018

Conversation

kurdybacha
Copy link
Contributor

Solution: libzmq pkg-config build (using autogen) added to Travis CI.

There is an issue with cppzmq where it needs to be installed as well in order to find pkg-config version of libzmq. That will be addressed in separate PR.

Solution: build and install libzmq using autogen tools.
Solution: Move sudo required build to top to start sooner to get better
overall build time.
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.
Solution: simplify Travis config file by defining default build options
DRAFT and ZMQ_VERSION in ci_build.sh script
@coveralls
Copy link

coveralls commented May 16, 2018

Pull Request Test Coverage Report for Build 94

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 25 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-45.3%) to 54.688%

Files with Coverage Reduction New Missed Lines %
zmq.hpp 25 61.97%
Totals Coverage Status
Change from base Build 93: -45.3%
Covered Lines: 35
Relevant Lines: 64

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 94

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 25 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-45.3%) to 54.688%

Files with Coverage Reduction New Missed Lines %
zmq.hpp 25 53.97%
Totals Coverage Status
Change from base Build 93: -45.3%
Covered Lines: 35
Relevant Lines: 64

💛 - Coveralls

@bluca
Copy link
Member

bluca commented May 16, 2018

You can install to a local directory and avoid the sudo

@kurdybacha
Copy link
Contributor Author

kurdybacha commented May 16, 2018

true, will change that. thanks.
ps please ignore coveralls reports - still showing random stuff.

@kurdybacha
Copy link
Contributor Author

@bluca, not that easy as I thought. for some reason cppzmq does not compile with locally installed pkg-config version of libzmq. I would need to spend more time on that some other day.

@sigiesec sigiesec merged commit b11e445 into zeromq:master May 17, 2018
@kurdybacha kurdybacha deleted the pkgconfig-build branch May 18, 2018 21:37
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 this pull request may close these issues.

None yet

4 participants