search

kevinkreiser / prime_server

non-blocking (web)server API for distributed computing and SOA based on zeromq
Other
59 stars 26 forks source link

Add cmake Build #68

Closed kevinkreiser closed 6 years ago

kevinkreiser commented 6 years ago

This fixes #67

Most of this work is from @PayasR. The main thing that made travis builds pass was making the include path be not just:

include_directories(${CMAKE_SOURCE_DIR}/prime_server)

But also just adding the root dir:

include_directories(${CMAKE_SOURCE_DIR} ${CMAKE_SOURCE_DIR}/prime_server)

I also removed the cmake find_package stuff using custom cmake modules and instead went back to using pkgconfig. I also removed the hardcoded pthread linker flag and got at pthread using find_package(Threads REQUIRED) and the variables it produces.

PayasR commented 6 years ago

Great, thanks! The README needs to be updated too though.

kevinkreiser commented 6 years ago

We're not ready to fully switch to cmake mainly because it's not what we are using for debian packaging. I haven't a full grasp on how to do that migration so until we fugure that out autotools is still the fool proof way of building the project. We probably should have more build config for Travis to build with autotools and cmake...