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.

Sign up for GitHub

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

Jump to bottom

Remove neopg-tool from install taget/cmake export #92

Merged

lambdafu merged 1 commit into das-labor:master from Bjoe:fix-cmake-export

Nov 10, 2018

Contributor

Bjoe commented Nov 9, 2018

There was a last missing review fix from #90 (comment)

And libneopg-tool exists only to allow for creating a separate testing binary that does unit testing with code from the main program.

With this PR, this review findings are also fixed. Libneopg-tool puls header files will not installed anymore with "install" target.


          Remove neopg-tool from export

8e2d594

Bjoe mentioned this pull request

Improve cmake build system #90

Merged

lambdafu merged commit ffcb56b into das-labor:master

Collaborator

lambdafu commented Nov 10, 2018

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet