920908 - Use EXPAND_PATH_LIBNAME when linking against libxul/libmozalloc

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

Currently, we're linking against libxul and libmozalloc with the "usual" -Lpath -llib arguments. Except on some platforms (mainly, windows). For various reasons, and to make things better for bug 905973, I want to use EXPAND_PATH_LIBNAME instead. Expandlibs_exec will do the right thing and still use the dynamic library.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Use EXPAND_PATH_LIBNAME when linking against libxul/libmozalloc

While I was here, I removed DYNAMIC_XPCOM_LIBS, which is not used anymore, and both DYNAMIC_XPCOM_LIBS and XPCOM_FROZEN_LDOPTS, that are not used at all in js/src. I also changed expandlibs_exec to use makeutil, and to *not* expose the dynamic libraries as dependencies. This matches the current behaviour. There is, however, an additional dummy *_order_only rule added for these dependencies, which is not directly useful, but will be indirectly used in bug 905973.

Note that isDynamicLib won't do the right thing on windows with import libs, which have the same extension as static libs, but this, too, matches the current behavior.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Use EXPAND_PATH_LIBNAME when linking against libxul/libmozalloc

Fixed a stupid typo.

Comment 3

[Gregory Szorc [:gps]]

Comment on attachment 810451 [details] [diff] [review]
Use EXPAND_PATH_LIBNAME when linking against libxul/libmozalloc

Review of attachment 810451 [details] [diff] [review]:
-----------------------------------------------------------------

Seeing a bunch of hardcoded foo in configure.in replaced with dynamic lookup makes me happy. I think.

I suppose this increases the risk we accidentally add a new library dependency to libxul, does it not? Conceivably it's possible we start linking against a new non-shipped library, don't notice, and builds blow up. I'm trying to think if that's a valid concern or not.

Comment 4

[Gregory Szorc [:gps]]

Comment on attachment 810451 [details] [diff] [review]
Use EXPAND_PATH_LIBNAME when linking against libxul/libmozalloc

Review of attachment 810451 [details] [diff] [review]:
-----------------------------------------------------------------

There weren't any -L arguments in the configure variables before. I was worried about shared libraries, not static. Derp.

Comment 5

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c6cb5b72f8d6

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c6cb5b72f8d6