On 2024/06/26 16:04, Sergey A. Osokin wrote: > Hi Stuart, > > thanks for the update. > > On Wed, Jun 26, 2024 at 04:23:17PM +0100, Stuart Henderson wrote: > > On 2024/06/01 18:30, Sergey A. Osokin wrote: > > > > > > here's the patch to update www/nginx third-party modules to their > > compiled as part of nginx itself and not in standalone separate ports). > > [...] > > > - mentions the actual modules used in COMMENT-lua to make it obvious > > that -lua needs a bump if ngx_devel_kit is changed > > The update of the third-party lua-nginx-module to its recent version > requires two additional ports to run nginx with the module: > - lua-resty-core > - lua-resty-lrucache > > Also, since the third-party lua-nginx-module is the part of the > OpenResty distribution, it's probably better to build lua-nginx-module > against luajit2, https://github.com/openresty/luajit2. That means > another port is required. So, I kept that part as is. > > > unless there are objections I'd like to commit in a few days as most > > of this has been sitting around for ages. (or earlier if ok'd ;) > > > > Not included in this diff but want looking at sometime; > > > > - LIB_DEPENDS is messed up for njs ("LIB_DEPENDS devel/pcre2 not needed for > > www/nginx,-njs" which means that pcre2 is stripped from the dependencies > > for that subpackage - looks like it only uses pcre 1.x?) > > njs module supports both pcre1 and pcre2, possible to allow one of > those libraries, and disallow another one. I'd prefer to see pcre2, > cause pcre1 is out of support for years.
The binary produced by this port is linked against pcre1. > > - other modules want updates: > > - headers-more 0.37 > > - lua-nginx-module 0.10.26 > > - naxsi https://github.com/wargio/naxsi 1.6 > > - ngx_http_hmac_secure_link_module > > dc641de6c2ff2dbdaae6c27073f77a3b36fd6d40 > > - (nginx-auth-ldap 241200eac8e4acae74d353291bd27f79e5ca3dc4 but only > > spelling changes) > > Thank you. > > -- > Sergey A. Osokin >