Le Thu, Sep 07, 2023 at 02:47:58PM +0000, Sergey A. Osokin a écrit : > 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.
right. then ... i think BUILD_DEPENDS + RUN_DEPENDS is needed in the static case instead. you might want to check WANTLIB too (make port-lib-depends-check), here it complains about missing xml2/z Landry
