Fixes to issue #244: libjson-rpc-cpp does not build on Linux. (#246)

marcosszydlo · cinemast · commit 56b3e0473f76 · 2018-08-23T23:11:51.000+02:00
* Fixes to issue #244: libjson-rpc-cpp does not build on Linux. There are no code changes made. Three files were changed: CMakeDependencies.cmake was missing the command find_package(Threads REQUIRED) The test file test_integration.cpp was missing an include file: <unistd.h> The third file that I modified, CMakeLists.txt, should be restored to its original form. I changed it so I could build withou HTTP, which I don't need. The code was built and tested on RedHat REL 7 (with HTTP disabled in CMake). * Re-enable HTTP libs
diff --git a/cmake/CMakeDependencies.cmake b/cmake/CMakeDependencies.cmake
@@ -38,6 +38,7 @@ if (${REDIS_SERVER} OR ${REDIS_CLIENT})
     message(STATUS "Hiredis lib   : ${HIREDIS_LIBRARIES}")
 endif()
 
+find_package(Threads REQUIRED)
 find_package(Doxygen)
 
 if (${COMPILE_TESTS})
diff --git a/src/test/test_integration.cpp b/src/test/test_integration.cpp
@@ -136,6 +136,7 @@ TEST_CASE("test_integration_unixdomain", TEST_MODULE) {
 
 #include <jsonrpccpp/client/connectors/filedescriptorclient.h>
 #include <jsonrpccpp/server/connectors/filedescriptorserver.h>
+#include <unistd.h>
 
 TEST_CASE("test_integration_filedescriptor", TEST_MODULE) {
   int c2sfd[2]; // Client to server fd
