On Thu, Sep 07, 2023 at 12:16:39PM +0200, Landry Breuil wrote:
> Le Wed, Sep 06, 2023 at 02:12:20PM +0000, Sergey A. Osokin a écrit :
[...]
> > 
> > Sure, here's the updated patch.
> 
> i've checked and something is wrong somewhere, because njs is detected
> at configure, the unitd binary is supposedly linked against it
> 
> cc -Wl,-E -L/usr/local/lib -o build/sbin/unitd -pipe -fPIC
> -fvisibility=hidden -W -Wall -Wextra -Wno-unused-parameter
> -Wwrite-strings -fstrict-aliasing -Wstrict-overflow=5
> -Wmissing-prototypes  -O2 -pipe -g -I/usr/local/include
> build/src/nxt_main.o   build/lib/libnxt.a  -lm -lpthread
> -L/usr/local/lib -Wl,-R/usr/local/lib -lnjs -lm -lpcre2-8 -lcrypto
> -lxml2 -lz -lssl -lcrypto    -L/usr/local/lib -Wl,-R/usr/local/lib
> -lpcre2-8
> 
> but packaging fails:
> Create /usr/ports/packages/amd64/all/unit-1.31.0p0.tgz
> LIB_DEPENDS lang/njs not needed for www/unit/unit ?
> 
> and looking at the binary the dependency on the njs library isnt
> registered
> 
> $objdump -p build/sbin/unitd|grep NEED
>   NEEDED      libm.so.10.1
>   NEEDED      libpthread.so.27.1
>   NEEDED      libpcre2-8.so.0.6
>   NEEDED      libcrypto.so.52.0
>   NEEDED      libxml2.so.19.0
>   NEEDED      libz.so.7.0
>   NEEDED      libssl.so.55.0
>   NEEDED      libc.so.97.1

There's no shared library for NJS at the moment, only static
library is available.

-- 
Sergey A. Osokin

Reply via email to