Le Thu, Oct 26, 2023 at 01:38:14PM +0000, Sergey A. Osokin a écrit :
> Hi Landry,
> 
> On Thu, Oct 26, 2023 at 08:51:19AM +0200, Landry Breuil wrote:
> > Le Wed, Oct 25, 2023 at 04:47:39PM +0000, Sergey A. Osokin a écrit :
> > > Hi,
> > > 
> > > here's the patch to update lang/njs to the recent version, 0.8.2.
> > > Also, the patch provides an update for njs module for www/nginx.
> > 
> > that patch only updates the version in nginx afaict..
> 
> here we go.

asking for the second or third time.. would it be possible to fix
upstream so that it can build itself if the previous version is
installed ? otherwise it still fails:

src/njs_flathsh.c:166:1: error: no previous prototype for function 
'njs_flathsh_destroy' [-Werror,-Wmissing-prototypes]
njs_flathsh_destroy(njs_flathsh_t *fh, njs_flathsh_query_t *fhq)

just because the order of includes is wrong.. '-I/usr/local/include  -Isrc  
-Ibuild  -I/usr/local/include/libxml2 -I/usr/local/include'

that doesnt happen when building the nginx njs module which uses '-I 
/usr/local/include/libxml2 -I /usr/local/include -Isrc -Ibuild -Injs'

would be nice :)

commited, thanks!

Reply via email to