Le Wed, Sep 06, 2023 at 02:12:20PM +0000, Sergey A. Osokin a écrit : > On Wed, Sep 06, 2023 at 05:13:24AM +0200, Landry Breuil wrote: > > Le Tue, Sep 05, 2023 at 05:57:09PM +0000, Sergey A. Osokin a écrit : > > > Landry, > > > > > > the recent update in lang/njs you made includes a fix for the > > > njs.pc, so we may have to return to this one? Or am I missing > > > something? > > > > maybe, dont remember the context :) > > > > in all cases it needs REVISION bumps since it changes the dependencies.. > > 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 that's .. puzzling. is anyone seeing the same failure ? Landry
