[Pkg-nginx-maintainers] nginx - experimental 1.22.0-2~exp3

[Jan Mojzis]

OK,
I will proceed more carefully, in smaller steps.

I've  prepared the 1.22.0-2~exp4 version, which has been in experimental for a week.
This version:
- adds libnginx-mod-http-ndk-dev package, which is needed to build the lua module
- contains a couple of simple bugfixes

I would like to merge these changes into the master and upload them to unstable.
MR here: https://salsa.debian.org/nginx-team/nginx/-/merge_requests/35 <https://salsa.debian.org/nginx-team/nginx/-/merge_requests/35>
Will need DD, because of new package libnginx-mod-http-ndk-dev.

After uploading, we will have a version of nginx, with which we will be able to build any module (including lua) separately.

Jan

> On 22. 7. 2022, at 19:06, Thomas Ward <teward at thomas-ward.net> wrote:
> 
> If we remove the Lua module, it will break existing installations that depend on Lua.  Adding the separate Lua module back in later will result in some... problems.
> 
> If we go and do a separate lua packaging as well, we need to go through ITP and NEW again for a separate source package, and we need to make sure that the current NGINX maintainers are kept identical in the new packages as well.  We will also need a DD to give us dak permissions for ftp-master upload permissions too.
> 
> I'm not against removal of the Lua module, but if we're going to do that, we're going to have to do so carefully.
> 
> 
> 
> Thomas
> 
> 
> 
> On 7/22/22 13:01, Jan Mojzis wrote:
>> Hello,
>> I am sending information about the conclusion from the last experimental version 1.22.0-2~exp3.
>> The goal of the testing was to verify how a separate lua module works.
>> And the second was to test the migration to the PCRE2 library.
>> 
>> Good news:
>> - The standalone lua module works as expected, thanks nginx-dev and  libnginx-mod-http-ndk-dev,
>> testing pipeline here: https://salsa.debian.org/janmojzis/ngx-lua/-/pipelines/402864 <https://salsa.debian.org/janmojzis/ngx-lua/-/pipelines/402864> (piuparts, reprotest failed, but it's not problem with package)
>> - The migration to PCRE2 library work for NGINX
>> 
>> Bad news:
>> - The migration to PCRE2 library doesn't work for lua even in the latest lua version.
>> e.g. issue: https://github.com/openresty/lua-nginx-module/issues/1984 <https://github.com/openresty/lua-nginx-module/issues/1984>
>> This is big issue,
>> and unfortunately the fix is not simple :(
>> 
>> So, in the next version, I propose to remove the lua module,
>> just like the Ubuntu developers did.
>> When lua maintainer fixes pcre2 problems, we will be ready to release new separate ngx-lua package.
>> 
>> If it's ok, I will prepare exp4, which will have this change, plus I would solve a couple of issues, e.g. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=876365 <https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=876365>
>> 
>> Jan	
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-nginx-maintainers/attachments/20220730/5036f394/attachment.htm>