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: Travis requires sudo and draft not enabled for git repo #240

Merged
merged 3 commits into from
May 23, 2018

Conversation

kurdybacha
Copy link
Contributor

Problem: Travis requires sudo for for pkg-config libzmq build as cppzmq can not
find installed pkg-config libzmq in custom location.

Solution: cppzmq to use cmake prefix paths for pkg config paths.

Problem: 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

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
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.
@coveralls
Copy link

coveralls commented May 23, 2018

Pull Request Test Coverage Report for Build 108

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

Files with Coverage Reduction New Missed Lines %
zmq.hpp 11 17.72%
Totals Coverage Status
Change from base Build 103: -10.4%
Covered Lines: 120
Relevant Lines: 177

💛 - Coveralls

@bluca bluca merged commit 1f14f9a into zeromq:master May 23, 2018
@kurdybacha kurdybacha deleted the draft-git branch June 4, 2018 15:59
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

3 participants