On Thu, Oct 26, 2023 at 03:53:07PM +0200, Landry Breuil wrote:
[...]
>
> 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 ?
I'm working on that.
> 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 :)
I attached a small patch from the lang/njs/patches directory,
that works for me. So, REVISION for the lang/njs/Makefile needs
to be bumped to `0'.
> commited, thanks!
Thank you!
--
Sergey A. Osokin
Index: auto/make
--- auto/make.orig
+++ auto/make
@@ -85,8 +85,9 @@ do
cat << END >> $NJS_MAKEFILE
$NJS_BUILD_DIR/$njs_obj: $njs_src
- \$(NJS_CC) -c \$(NJS_CFLAGS) $NJS_LIB_AUX_CFLAGS \\
+ \$(NJS_CC) -c \$(NJS_CFLAGS) \\
\$(NJS_LIB_INCS) -Injs \\
+ $NJS_LIB_AUX_CFLAGS \\
-o $NJS_BUILD_DIR/$njs_obj \\
$njs_dep_flags \\
$njs_src
