<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">OK,<br class="">I will proceed more carefully, in smaller steps.<div class=""><br class=""></div><div class="">I've prepared the 1.22.0-2~exp4 version, which has been in experimental for a week.<br class="">This version:<br class="">- adds libnginx-mod-http-ndk-dev package, which is needed to build the lua module</div><div class="">- contains a couple of simple bugfixes</div><div class=""><br class=""></div><div class="">I would like to merge these changes into the master and upload them to unstable.</div><div class="">MR here: <a href="https://salsa.debian.org/nginx-team/nginx/-/merge_requests/35" class="">https://salsa.debian.org/nginx-team/nginx/-/merge_requests/35</a></div><div class="">Will need DD, because of new package libnginx-mod-http-ndk-dev.</div><div class=""><br class=""></div><div class="">After uploading, we will have a version of nginx, with which we will be able to build any module (including lua) separately.</div><div class=""><br class=""></div><div class="">Jan<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 22. 7. 2022, at 19:06, Thomas Ward <<a href="mailto:teward@thomas-ward.net" class="">teward@thomas-ward.net</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" class="">
<div class="">
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.<br class="">
<br class="">
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.<br class="">
<br class="">
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.<br class="">
<br class="">
<br class="">
<br class="">
Thomas<br class="">
<br class="">
<br class="">
<br class="">
<div class="moz-cite-prefix">On 7/22/22 13:01, Jan Mojzis wrote:<br class="">
</div>
<blockquote type="cite" cite="mid:53D9BD09-0603-47BB-B22E-3CCF3FFD42D2@gmail.com" class="">
Hello,
<div class="">I am sending information about the conclusion from
the last experimental version 1.22.0-2~exp3.<br class="">
The goal of the testing was to verify how a separate lua module
works.<br class="">
And the second was to test the migration to the PCRE2 library.</div>
<div class=""><br class="">
</div>
<div class="">Good news:</div>
<div class="">- The standalone lua module works as expected,
thanks nginx-dev and libnginx-mod-http-ndk-dev,</div>
<div class="">testing pipeline here: <a href="https://salsa.debian.org/janmojzis/ngx-lua/-/pipelines/402864" class="moz-txt-link-freetext" moz-do-not-send="true">https://salsa.debian.org/janmojzis/ngx-lua/-/pipelines/402864</a> (piuparts,
reprotest failed, but it's not problem with package)</div>
<div class="">- <span style="caret-color: rgb(0, 0, 0);" class="">The migration to PCRE2 library</span> work
for NGINX</div>
<div class=""><br class="">
</div>
<div class="">Bad news:</div>
<div class="">- <span style="caret-color: rgb(0, 0, 0);" class="">The migration to PCRE2 library</span><font class=""><span style="caret-color: rgb(0, 0,
0);" class=""> doesn't work for lua even in the latest lua
version.</span></font></div>
<div class=""><font class=""><span style="caret-color: rgb(0, 0, 0);" class="">e.g. issue: <a href="https://github.com/openresty/lua-nginx-module/issues/1984" class="moz-txt-link-freetext" moz-do-not-send="true">https://github.com/openresty/lua-nginx-module/issues/1984</a></span></font></div>
<div class=""><font class=""><span style="caret-color: rgb(0, 0, 0);" class="">This is big
issue,</span></font></div>
<div class="">and unfortunately the fix is not simple :(</div>
<div class=""><br class="">
</div>
<div class="">So, in the next version, I propose to remove the lua
module,<br class="">
just like the Ubuntu developers did.</div>
<div class="">When lua maintainer fixes pcre2 problems, we will be
ready to release new separate ngx-lua package.</div>
<div class=""><br class="">
</div>
<div class="">If it's ok, I will prepare exp4, which will have
this change, plus I would solve a couple of issues, e.g. <a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=876365" class="moz-txt-link-freetext" moz-do-not-send="true">https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=876365</a></div>
<div class=""><br class="">
</div>
<div class="">Jan<span class="Apple-tab-span" style="white-space:pre"> </span></div>
</blockquote>
<br class="">
</div>
</div></blockquote></div><br class=""></div></body></html>