714029 - Avoid jsctypes-test dependency on the xpcom glue

Status: RESOLVED FIXED

(Core :: js-ctypes, defect)

Description

[Mike Hommey [:glandium]]

The jsctypes-test uses NS_strlen, which is in the xpcom glue. It links against the standalone glue for that. For purpose of bug 683127 testing, the xpcom standalone glue is modified to use wrapped calls to dlopen and related functions in the part that initializes xpcom dynamically. However, due to linker stupidity, despite that part being dead code, and despite the linker not including it in the library, it still wants to resolve the dlopen and related wrappers, and thus fails to link.
Since jsctypes-test only uses NS_strlen, it's actually pretty straightforward to just get rid of the whole dependency.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Avoid jsctypes-test dependency on the xpcom glue

See comment 0 for the stupid reasons why this is needed. Since I was adding a simple implementation of NS_strlen, I implemented it as a template and replaced plain strlen as well.

Comment 2

[Mike Hommey [:glandium]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/057e7ead186e

Comment 3

[Mike Hommey [:glandium]]

Backed out, it broke windows. Looks like a missing mozalloc on the link command line.
https://hg.mozilla.org/integration/mozilla-inbound/rev/b2447177ec5a

Comment 4

[Mike Hommey [:glandium]]

Relanded with a fixup to keep MOZALLOC_LIB in EXTRA_DSO_LDOPTS
https://hg.mozilla.org/integration/mozilla-inbound/rev/e9aeacb404ba

Comment 5

[Ed Morley [:emorley]]

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

Comment 6

[Mike Hommey [:glandium]]

Attachment: Avoid jsctypes-test dependency on the xpcom glue - for aurora

[Approval Request Comment]
This is a dependency of bug 683127. It changes how the js-ctypes test library is linked (removes the dependency on the xpcom glue and library dependencies it doesn't need)

Comment 7

[Mike Hommey [:glandium]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/d8158788ac4d