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

Daniel-Constantin Mierla notifications at github.com
Wed Feb 14 11:33:39 CET 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.kamailio.org/pipermail/sr-dev/attachments/20180214/b54e2d0c/attachment.html>


More information about the sr-dev mailing list