speeding up configure (was: [Bug 959] configure adds -lX11 without checking)

Witold Filipczyk writes:

> On my branch I made that check and other changes to
> configure.in which significantly speed up builds on old
> hardware.

What kind of hardware and how many seconds of speedup?

My login script sets CPPFLAGS="-I${HOME}/prefix/include" and
LDFLAGS="-L${HOME}/prefix/${machtype}/lib", and I install SEE and
FSP in those directories.  elinks/configure --help indicates that
these environment variables can be used for specifying
nonstandard locations of headers and libraries.  Previously, the
SpiderMonkey check also respected these variables: the second
iteration of the outer loop has spidermonkeydir="", and that
value made the script try to use the library without adding any
-I or -L options.  It seems you have now removed this
possibility, by making the script insist on test -f
"$spidermonkeydir$spidermonkeyinclude/jsapi.h".

On the other hand, I do not know whether SpiderMonkey really
supports being installed and used in that way.  The install
target in xulrunner-1.8.1.4/config/rules.mk does appear to create
a $(MODULE) subdirectory in $(includedir), and the JavaScript C
Engine Embedder's Guide talks about including "jsapi.h" and not
"mozjs/jsapi.h", so perhaps SpiderMonkey expects to always get a
separate -I option.

I am thus not entirely against such a change.  However, if
spidermonkeydir="" is now intended to mean the root directory
rather than the default, I think the test ! -z "$spidermonkeydir"
should then be removed, so that the script always adds the -I and
-L options.

I have not reviewed the Lua change yet.

_______________________________________________
elinks-dev mailing list
[email protected]
http://linuxfromscratch.org/mailman/listinfo/elinks-dev

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFGjYtKHm9IGt60eMgRApr8AKCyxWG0oiyV+CVjXqF/4iEdpI2DzACgkNWv
FdFM38iyOYh1i0bfjqm5GiQ=
=qJ67
-----END PGP SIGNATURE-----