Re: [sr-dev] [kamailio/kamailio] db_redis: Implement db_redis generic db driver (#1432)

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 or view it on GitHub: https://github.com/kamailio/kamailio/pull/1432#issuecomment-365562086