svn commit: r512184 - in head/www/nginx-devel: . files

[Herbert J. Skuhra]

On Fri, Sep 20, 2019 at 12:29:08PM +0000, Sergey A. Osokin wrote:
> On Thu, Sep 19, 2019 at 09:41:33PM +0200, Herbert J. Skuhra wrote:
> > On Mon, 16 Sep 2019 22:38:21 +0200, "Sergey A. Osokin" wrote:
> > > 
> > > Author: osa
> > > Date: Mon Sep 16 20:38:21 2019
> > > New Revision: 512184
> > > URL: https://svnweb.freebsd.org/changeset/ports/512184
> > > 
> > > Log:
> > >   Add the following third-party modules:
> > >   o) ip2location
> > >   o) ip2proxy
> > >   
> > >   Add third-party slice_ahead feature.
> > >   
> > >   Update third-party passenger module to the latest release 6.0.3.
> > >   
> > >   Switch third-party lua module dependence from lang/luajit (2.0) to
> > >   lang/luajit-openresty (2.1).
> > >   
> > >   Bump PORTREVISION.
> > > 
> > > Modified:
> > >   head/www/nginx-devel/Makefile
> > >   head/www/nginx-devel/Makefile.extmod
> > >   head/www/nginx-devel/Makefile.options.desc
> 
> > >   head/www/nginx-devel/distinfo
> > >   head/www/nginx-devel/files/extra-patch-passenger-build-nginx.rb
> > >   head/www/nginx-devel/files/extra-patch-passenger-disable-telemetry
> > 
> > This commit is obviously wrong/incomplete:
> > 
> > 1. option HTTP_SLICE_AHEAD was added in wrong place
> > 2. extra-patch-src_http_modules_ngx_http_slice_read_ahead.c is missing
> 
> Hi Herbert,
> 
> thanks for point me in.  The missing patch has been found and added
> to the port.
> Could you please explain what does it mean HTTP_SLICE_AHEAD in a wrong
> place.

Hi Sergey,

thanks.

HTTP_SLICE_AHEAD is an external module but the option was added to
"Modules that are part of the base nginx distribution".

-- 
Herbert