add pcre build support for alpine & alpine-fat

[chrisLeeTW]

Hello,

I've faced the same problem for this issue (openresty/lua-resty-core#258) with alpine & alpine-fat.

It's looks like there no pcre support during the compilation.

May I know that is intended behavior or just some thing missing before ?

Thanks.

[neomantra]

We install PCRE ourselves as is needed for the OpenResty build, but maybe we need to do what agentzh refers to here. I will research it further.

[vkill]

+1

[neomantra]

@vkill Thanks for the PR. There's also the Nginx compile lines in that link I have above... I'll pow wow with the OpenResty packagers this week to nail down the exact incantations needed.

I'm worried that what you have in the PR fixes one thing but doesn't fix others. But I also wonder if your PR also fixes #22, which would be great.

I also want to fix all the build-from-source images in one swoop, so will expand it to that.

[vkill]

Hi @neomantra , #22 was fixed. the --enable-utf --enable-unicode-properties options make it effective. And the options same as openresty travis.yml .

The nginx.conf

worker_processes  1;

events {
    worker_connections 4;
}

http {
    server {
        listen 81;

        location /re {
            content_by_lua_block {
                local ngx_re = require "ngx.re"

                local res, err = ngx_re.split("a,b,c,d", ",")
                if err then
                    ngx.log(ngx.ERR, "failed: ", err)
                    return
                end

                for i = 1, #res do
                    ngx.say(res[i])
                end

                local newstr, n, err = ngx.re.gsub("测 试 用 例", "[ \t]", "", "ju")
                ngx.say(newstr)
            }
        }
    }
}

The result

$ docker run --name or_re_test_2 --rm -p 9002:81 -v $(pwd)/nginx.conf:/usr/local/openresty/nginx/conf/nginx.conf vkill/openresty:alpine-with-pcre
$ curl http://127.0.0.1:9002/re

a
b
c
d
测试用例

The ldd result

/ # ldd /usr/local/openresty/nginx/sbin/nginx 
	/lib/ld-musl-x86_64.so.1 (0x7fda9d2b5000)
	libluajit-5.1.so.2 => /usr/local/openresty/luajit/lib/libluajit-5.1.so.2 (0x7fda9ccbd000)
	libpcre.so.1 => /usr/local/lib/libpcre.so.1 (0x7fda9cc49000)
	libz.so.1 => /lib/libz.so.1 (0x7fda9cc2f000)
	libc.musl-x86_64.so.1 => /lib/ld-musl-x86_64.so.1 (0x7fda9d2b5000)
	libgcc_s.so.1 => /usr/lib/libgcc_s.so.1 (0x7fda9cc1b000)

[neomantra]

@vkill @chrisLeeTW I've just pushed a modified version of the #113 pull request. Please check it out and let me know if it works for you. I'm adding one more change to this release and then will tag it. Thanks for the testing and the direction for fixing this issue.

@vkill I'm going to close your PR once this is released but if you want to be in the AUTHORS file, give me the name+email you want in there (or no email is fine too).

Once the automated build is done, these will be at the simple alpine and alpine-fat image tags.

[vkill]

@neomantra It works for me, and I was built the new image for apisix, it also works.

[neomantra]

Great, I just tagged 1.15.8.1-3 with this. Thanks again for the help.