Skip to content

Conversation

sisco0
Copy link

@sisco0 sisco0 commented Sep 26, 2021

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
Copy link
Contributor

hzeller commented Sep 26, 2021

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
Copy link
Author

sisco0 commented Sep 26, 2021

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

@sisco0 sisco0 closed this Sep 26, 2021
@cinemast
Copy link
Owner

Thanks to all of you!

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

Successfully merging this pull request may close these issues.

3 participants