请agentzh给 openresty中的nginx加个补丁

lancelijade · 2013-12-06T13:29:08+00:00

是的，看错了，你用的是print 不是ngx.print()理解&#...

请agentzh给 openresty中的nginx加个补丁

lancelijade

hi agentzh,

请给 openresty 中的 nginx 添加一个补丁以激活 pcre-utf 支持，因为标准的 nginx 里的模板匹配似乎不支持 utf 模式，但 ngx_lua 的 re.* 是支持 u 模式的。

具体方法可以在 configure 时添加 --with-pcre-utf 选项，然后给 nginx/auto/lib/pcre/conf 打补丁实现。

谢谢

Lance

agentzh

Hello!

2013/12/5 Lance:
> 请给 openresty 中的 nginx 添加一个补丁以激活 pcre-utf 支持，因为标准的 nginx 里的模板匹配似乎不支持 utf 模式，但
> ngx_lua 的 re.* 是支持 u 模式的。
>
> 具体方法可以在 configure 时添加 --with-pcre-utf 选项，然后给 nginx/auto/lib/pcre/conf 打补丁实现。
>

欢迎贡献这样的补丁 :) 我现在实在太忙了 ;)

不过我觉得添加一个更一般的 --with-pcre-config-opt=OPTIONS 选项或许更有用一些？

Regards,
-agentzh

lancelijade

好主意

不过这样的补丁应该加在哪里？好象 openresty 本身不是一个 git 项目吧

Lance

2013/12/6 Yichun Zhang (agentzh) <age...@gmail.com>

Hello!

2013/12/5 Lance:

> 请给 openresty 中的 nginx 添加一个补丁以激活 pcre-utf 支持，因为标准的 nginx 里的模板匹配似乎不支持 utf 模式，但
> ngx_lua 的 re.* 是支持 u 模式的。
>
> 具体方法可以在 configure 时添加 --with-pcre-utf 选项，然后给 nginx/auto/lib/pcre/conf 打补丁实现。
>

欢迎贡献这样的补丁 :) 我现在实在太忙了 ;)

不过我觉得添加一个更一般的 --with-pcre-config-opt=OPTIONS 选项或许更有用一些？

Regards,
-agentzh

--

agentzh

Hello!

2013/12/5 Lance:
>
> 不过这样的补丁应该加在哪里？好象 openresty 本身不是一个 git 项目吧
>

是一个 git 项目，nginx 相关的补丁都放在这里：

  https://github.com/agentzh/ngx_openresty/tree/master/patches/

你只需提供 nginx 1.4.3 对应的补丁文件就可以了。然后在 util/mirror-tarballs 脚本中应用此补丁。

-agentzh

lancelijade

git pull request 没搞定，直接把patch放在这里了

Lance

2013/12/6 Yichun Zhang (agentzh) <age...@gmail.com>

Hello!

2013/12/5 Lance:

>
> 不过这样的补丁应该加在哪里？好象 openresty 本身不是一个 git 项目吧
>

是一个 git 项目，nginx 相关的补丁都放在这里：

https://github.com/agentzh/ngx_openresty/tree/master/patches/

你只需提供 nginx 1.4.3 对应的补丁文件就可以了。然后在 util/mirror-tarballs 脚本中应用此补丁。

-agentzh

--

Attachment: nginx-1.4.3-with_pcre_config_opt.patch
Description: Binary data

smallfish.xy

自己fork下分支，然后把patch提交到自己分支里。

然后在agentzh分支里create pullrequest，会有选择版本的

smallfish http://chenxiaoyu.org

2013/12/6 Lance <lance...@gmail.com>

git pull request 没搞定，直接把patch放在这里了

Lance

2013/12/6 Yichun Zhang (agentzh) <age...@gmail.com>
Hello!

2013/12/5 Lance:

>
> 不过这样的补丁应该加在哪里？好象 openresty 本身不是一个 git 项目吧
>

是一个 git 项目，nginx 相关的补丁都放在这里：

https://github.com/agentzh/ngx_openresty/tree/master/patches/

你只需提供 nginx 1.4.3 对应的补丁文件就可以了。然后在 util/mirror-tarballs 脚本中应用此补丁。

-agentzh

--

--

lancelijade

搞定了，我本地的 git push 不上去，之前在线编辑的没搞定，刚刚弄明白了

Lance

2013/12/8 smallfish <small...@gmail.com>

自己fork下分支，然后把patch提交到自己分支里。

然后在agentzh分支里create pullrequest，会有选择版本的

--
smallfish http://chenxiaoyu.org

2013/12/6 Lance <lance...@gmail.com>

git pull request 没搞定，直接把patch放在这里了

Lance

2013/12/6 Yichun Zhang (agentzh) <age...@gmail.com>
Hello!

2013/12/5 Lance:

>
> 不过这样的补丁应该加在哪里？好象 openresty 本身不是一个 git 项目吧
>

是一个 git 项目，nginx 相关的补丁都放在这里：

https://github.com/agentzh/ngx_openresty/tree/master/patches/

你只需提供 nginx 1.4.3 对应的补丁文件就可以了。然后在 util/mirror-tarballs 脚本中应用此补丁。

-agentzh

--

--

--

agentzh

Hello!

2013/12/6 Lance:
> git pull request 没搞定，直接把patch放在这里了
>

多谢，多谢！不过貌似这个补丁可以更简单一些，你觉得我弄的下面这个补丁如何？

diff -r 30e806b8636a auto/options
--- a/auto/options Mon Dec 09 10:16:44 2013 +0400
+++ b/auto/options Mon Dec 09 21:32:52 2013 -0800
@@ -286,6 +286,7 @@
         --with-pcre)                     USE_PCRE=YES               ;;
         --with-pcre=*)                   PCRE="$value"              ;;
         --with-pcre-opt=*)               PCRE_OPT="$value"          ;;
+        --with-pcre-conf-opt=*)          PCRE_CONF_OPT="$value"     ;;
         --with-pcre-jit)                 PCRE_JIT=YES               ;;

         --with-openssl=*)                OPENSSL="$value"           ;;
@@ -441,6 +442,7 @@
   --with-pcre                        force PCRE library usage
   --with-pcre=DIR                    set path to PCRE library sources
   --with-pcre-opt=OPTIONS            set additional build options for PCRE
+  --with-pcre-conf-opt=OPTIONS       set additional configure options for PCRE
   --with-pcre-jit                    build PCRE with JIT compilation support

   --with-md5=DIR                     set path to md5 library sources

diff -r 30e806b8636a auto/options
--- a/auto/options	Mon Dec 09 10:16:44 2013 +0400
+++ b/auto/options	Mon Dec 09 21:33:32 2013 -0800
@@ -286,6 +286,7 @@
         --with-pcre)                     USE_PCRE=YES               ;;
         --with-pcre=*)                   PCRE="$value"              ;;
         --with-pcre-opt=*)               PCRE_OPT="$value"          ;;
+        --with-pcre-conf-opt=*)          PCRE_CONF_OPT="$value"     ;;
         --with-pcre-jit)                 PCRE_JIT=YES               ;;
 
         --with-openssl=*)                OPENSSL="$value"           ;;
@@ -441,6 +442,7 @@
   --with-pcre                        force PCRE library usage
   --with-pcre=DIR                    set path to PCRE library sources
   --with-pcre-opt=OPTIONS            set additional build options for PCRE
+  --with-pcre-conf-opt=OPTIONS       set additional configure options for PCRE
   --with-pcre-jit                    build PCRE with JIT compilation support
 
   --with-md5=DIR                     set path to md5 library sources

lancelijade

哦，直接改 PCRE_CONF_OPT 变量，确实更简单。我没通读配置，不确认这个变量在哪里生成的，所以弄了个新的。

Lance

On Tue, Dec 10, 2013 at 1:34 PM, Yichun Zhang (agentzh) <age...@gmail.com> wrote:

Hello!

2013/12/6 Lance:

> git pull request 没搞定，直接把patch放在这里了
>

多谢，多谢！不过貌似这个补丁可以更简单一些，你觉得我弄的下面这个补丁如何？

diff -r 30e806b8636a auto/options
--- a/auto/options Mon Dec 09 10:16:44 2013 +0400
+++ b/auto/options Mon Dec 09 21:32:52 2013 -0800
@@ -286,6 +286,7 @@
--with-pcre) USE_PCRE=YES ;;
--with-pcre=*) PCRE="$value" ;;
--with-pcre-opt=*) PCRE_OPT="$value" ;;
+ --with-pcre-conf-opt=*) PCRE_CONF_OPT="$value" ;;
--with-pcre-jit) PCRE_JIT=YES ;;

--with-openssl=*) OPENSSL="$value" ;;
@@ -441,6 +442,7 @@
--with-pcre force PCRE library usage
--with-pcre=DIR set path to PCRE library sources
--with-pcre-opt=OPTIONS set additional build options for PCRE
+ --with-pcre-conf-opt=OPTIONS set additional configure options for PCRE
--with-pcre-jit build PCRE with JIT compilation support

--with-md5=DIR set path to md5 library sources

--