search

cinemast / libjson-rpc-cpp

C++ framework for json-rpc (json remote procedure call)
MIT License
955 stars 320 forks source link

Fix building for newest g++ and CMake #309

Closed sisco0 closed 3 years ago

sisco0 commented 3 years ago

Some warnings appeared during the CMake build process, where the names of the packages were not correctly specified in its type casing and some policy was not needed. Those changes are reflected in the *.cmake modified files.

Added to that, some slight modifications on the return type for callback and header_iterator functions were required for accomplishing the compilation process; these changes were done as well.

Tests have passed successfully after running the build/bin/unit_testsuite file with a running Redis server in the localhost.

hzeller commented 3 years ago

note, the MHD_Result will only work in newer libmicrohttpd versions. In PR #308 this is fixed to work for old and new versions.

sisco0 commented 3 years ago

Thank you very much @hzeller . I would close this PR, yours takes into account a wider variety of environments.

cinemast commented 3 years ago

Thanks to all of you!