Made igbinary optional.

Author: nicolasff

README.markdown

@@ -60,6 +60,12 @@ Error handling
 
 phpredis throws a `RedisException` object if it can't reach the Redis server. That can happen in case of connectivity issues, if the Redis service is down, or if the redis host is overloaded. In any other problematic case that does not involve an unreachable server (such as a key not existing, an invalid command, etc), phpredis will return `FALSE`.
 
+Igbinary
+==============
+
+phpredis supports Igbinary serialization when configured with `--enable-redis-igbinary`. Igbinary can be installed from PECL: `sudo pecl install igbinary`. Igbinary is much more efficient than the standard PHP `serialize()`.
+
+
 Methods
 =========
 
@@ -151,7 +157,7 @@ Set client option.
 <pre>
 $redis->setOption(Redis::OPT_SERIALIZER, Redis::SERIALIZER_NONE);	// don't serialize data
 $redis->setOption(Redis::OPT_SERIALIZER, Redis::SERIALIZER_PHP);	// use built-in serialize/unserialize
-$redis->setOption(Redis::OPT_SERIALIZER, Redis::SERIALIZER_IGBINARY);	// use igBinary serialize/unserialize
+$redis->setOption(Redis::OPT_SERIALIZER, Redis::SERIALIZER_IGBINARY);	// use Igbinary serialize/unserialize, only if enabled in `.configure` script.
 
 $redis->setOption(Redis::OPT_PREFIX, 'myAppName:');	// use custom prefix on all keys
 </pre>

config.m4

@@ -5,7 +5,48 @@ PHP_ARG_ENABLE(redis, whether to enable redis support,
 dnl Make sure that the comment is aligned:
 [  --enable-redis           Enable redis support])
 
+PHP_ARG_ENABLE(redis-igbinary, whether to enable igbinary serializer support,
+[  --enable-redis-igbinary      Enable igbinary serializer support], no, no)
+
 if test "$PHP_REDIS" != "no"; then
+  if test "$PHP_REDIS_IGBINARY" != "no"; then
+    AC_MSG_CHECKING([for igbinary includes])
+    igbinary_inc_path=""
+
+    if test -f "$abs_srcdir/include/php/ext/igbinary/igbinary.h"; then
+      igbinary_inc_path="$abs_srcdir/include/php"
+    elif test -f "$abs_srcdir/ext/igbinary/igbinary.h"; then
+      igbinary_inc_path="$abs_srcdir"
+    elif test -f "$phpincludedir/ext/session/igbinary.h"; then
+      igbinary_inc_path="$phpincludedir"
+    else
+      for i in php php4 php5 php6; do
+        if test -f "$prefix/include/$i/ext/igbinary/igbinary.h"; then
+          igbinary_inc_path="$prefix/include/$i"
+        fi
+      done
+    fi
+
+    if test "$igbinary_inc_path" = ""; then
+      AC_MSG_ERROR([Cannot find igbinary.h])
+    else
+      AC_MSG_RESULT([$igbinary_inc_path])
+    fi
+  fi
+
+  AC_MSG_CHECKING([for redis igbinary support])
+  if test "$PHP_REDIS_IGBINARY" != "no"; then
+    AC_MSG_RESULT([enabled])
+    AC_DEFINE(HAVE_REDIS_IGBINARY,1,[Whether redis igbinary serializer is enabled])
+    IGBINARY_INCLUDES="-I$igbinary_inc_path"
+    ifdef([PHP_ADD_EXTENSION_DEP],
+    [
+      PHP_ADD_EXTENSION_DEP(redis, igbinary)
+    ])
+  else
+    IGBINARY_INCLUDES=""
+    AC_MSG_RESULT([disabled])
+  fi
 
   dnl # --with-redis -> check with-path
   dnl SEARCH_PATH="/usr/local /usr"     # you might want to change this
@@ -46,5 +87,5 @@ if test "$PHP_REDIS" != "no"; then
   dnl
   dnl PHP_SUBST(REDIS_SHARED_LIBADD)
 
-  PHP_NEW_EXTENSION(redis, redis.c library.c redis_session.c igbinary/igbinary.c igbinary/hash_si.c igbinary/hash_function.c, $ext_shared)
+  PHP_NEW_EXTENSION(redis, redis.c library.c redis_session.c, $ext_shared)
 fi