A quick note that I updated the path for including hiredis.h:

diff --git a/src/modules/db_redis/redis_connection.h b/src/modules/db_redis/redis_connection.h
index d6ed4e4ec3..35d58e46a4 100644
--- a/src/modules/db_redis/redis_connection.h
+++ b/src/modules/db_redis/redis_connection.h
@@ -23,7 +23,7 @@
 #ifndef _REDIS_CONNECTION_H_
 #define _REDIS_CONNECTION_H_
 
-#include <hiredis.h>
+#include <hiredis/hiredis.h>
 
 #include "db_redis_mod.h"

On Debian 9 is where the file is located based on pkg-config output, it doesn't compile otherwise (same on MacOS with macports).

I looked in ndb_redis module and has the same path. If you have it in your system working without <hiredis/hiredis.h>, then we need to do some defines to set the proper path so it works on common distros.

Thanks again for this contribution!

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.