Permalink
Browse files

added a patch for nginx 0.8.53 to make this module work with nginx 0.…

…8.42+.
  • Loading branch information...
agentzh committed Nov 7, 2010
1 parent 65b5b06 commit 64bf7b0f23672c3d52dd5a950a8243ce489da59c
Showing with 13 additions and 4 deletions.
  1. +8 −2 README
  2. +5 −2 util/build.sh
View
10 README
@@ -57,12 +57,18 @@ such that ngx_eval's filter works *after* your filter modules.
Compatibility
The following versions of Nginx should work with this module:
- * 0.8.0 ~ 0.8.41
+ * 0.8.0 ~ 0.8.41 (0.8.42+ requires patching, see below)
* 0.7.x >= 0.7.21 (last tested version is 0.7.66)
Note that nginx 0.8.42+ won't work due to a famous regression appeared
-since 0.8.42: <http://forum.nginx.org/read.php?29,103078,103078 >.
+since 0.8.42: <http://forum.nginx.org/read.php?29,103078,103078 >, but fortunately a patch is available for
+nginx 0.8.53:
+
+ http://agentzh.org/misc/nginx/nginx-0.8.53-rewrite_phase_fix.patch
+
+This one-line patch should also be able to apply cleanly to
+other versions of nginx 0.8.42+.
Original ngx_eval documentation:
View
@@ -25,8 +25,9 @@ if [ ! -s "nginx-$version.tar.gz" ]; then
fi
#tar -xzvf nginx-$version.tar.gz || exit 1
-#cp $root/../no-pool-nginx/nginx-0.8.41-no_pool.patch ./
-#patch -p0 < nginx-0.8.41-no_pool.patch
+#cp $root/../no-pool-nginx/nginx-$version-no_pool.patch ./ || exit 1
+#patch -p0 < nginx-$version-no_pool.patch || exit 1
+#patch -p0 < ~/work/nginx-$version-rewrite_phase_fix.patch || exit 1
cd nginx-$version/
@@ -36,7 +37,9 @@ if [[ "$BUILD_CLEAN" -eq 1 || ! -f Makefile || "$root/config" -nt Makefile || "$
--add-module=$root $opts \
--add-module=$root/../echo-nginx-module \
--add-module=$root/../memc-nginx-module \
+ --add-module=$root/../ndk-nginx-module \
--with-debug
+ #--add-module=$root/../vallery/eval-nginx-module \
#--add-module=$home/work/nginx/nginx_upstream_hash-0.3 \
#--without-http_ssi_module # we cannot disable ssi because echo_location_async depends on it (i dunno why?!)

0 comments on commit 64bf7b0

Please sign in to comment.