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

Update cmake build system #17

Open
ftynse opened this issue Nov 27, 2015 · 0 comments
Open

Update cmake build system #17

ftynse opened this issue Nov 27, 2015 · 0 comments

Comments

@ftynse
Copy link
Collaborator

ftynse commented Nov 27, 2015

With cmake, we compile all the sources twice: first, for the static library, and then, for the shared library. However, cmake 2.8.8 introduced object libraries to avoid such recompilation.

Debug/Release states are hardcoded in the current CMakeLists, but they should rather depend on CMAKE_BUILD_TYPE.

Finally, I do not see the reason behind installing the test suite to /usr/share.

See ftynse/chlore@39ade18 for example file.

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

No branches or pull requests

1 participant