rkd77 / elinks

Fork of elinks
Other
335 stars 35 forks source link

build failure of latest master (a92df85c11edc21dfe4ead5af3d4fc908329b862) #81

Open balducci opened 3 years ago

balducci commented 3 years ago

hi

just tried to build latest master (a92df85c11edc21dfe4ead5af3d4fc908329b862) and getting this failure:

----8<----
gcc -m64 -Isrc/elinks.p -Isrc -I../src -I. -I.. -I/opt/stow.d/stow/zlib/usr/include -I/opt/stow.d/versions/openssl-3.0.0-alpha8/usr/include -I/opt/stow.d/versions/expat-2.2.10/usr/include -I/opt/stow.d/versions/python3-3.9.0/usr/python3/include/python3.9 -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -g '-DGETTEXT_PACKAGE="elinks"' '-DBUILD_ID=""' -DHAVE_CONFIG_H -fno-strict-aliasing -Wno-pointer-sign -Wno-builtin-declaration-mismatch -Wno-address -MD -MQ src/elinks.p/dialogs_info.c.o -MF src/elinks.p/dialogs_info.c.o.d -o src/elinks.p/dialogs_info.c.o -c ../src/dialogs/info.c
In file included from ../src/dialogs/info.c:22:
../src/ecmascript/ecmascript.h:8:10: fatal error: jsapi.h: No such file or directory
    8 | #include <jsapi.h>
      |          ^~~~~~~~~
compilation terminated.
---->8----

Previous commit (085a93d54471d9430c9b18431da1bd1d387e8e98) builds just fine (as any other commit before that, since quite a bit)

Building with meson:

----8<----
--prefix=/opt/stow.d/versions/felinks-4321/usr
--libdir=/opt/stow.d/versions/felinks-4321/usr/lib64
-Dperl=false
-Dpython=true
-Dgnutls=false
-Dspidermonkey=false
-Dsm-scripting=false
-Dtre=false
-Dfsp=false
---->8----

The build failure is apparently due to this include:

----8<----
#include "ecmascript/ecmascript.h"
---->8----

in the following source files:

----8<----
src/main/module.c
src/document/renderer.c
src/viewer/text/link.c
src/viewer/text/vs.c
src/viewer/text/form.c
src/protocol/protocol.c
src/dialogs/info.c
---->8----

As a matter of fact, if I comment the ecmascript.h include in those files the build succeeds.

Shouldn't the ecmascript stuff be skipped when -Dspidermonkey=false and -Dsm-scripting=false?

Thanks for any help

ciao -gabriele

rkd77 commented 3 years ago

Sorry, try now.

balducci commented 3 years ago

everything is back on track thanks