Export to GitHub

tenfourfox - issue #266

implement ICU shared machinery when it becomes mandatory


Posted on Mar 23, 2014 by Massive Rhino

ld: bl out of range (-17524756 max is +/-16M) from _MOZ_ReportAssertionFailure at 0x049742E8 in __text_cold of ../../xpcom/base/ClearOnShutdown.o to _fprintf$LDBL128$stub$island$5 at 0x038BDB20 in __text of XUL in _MOZ_ReportAssertionFailure from ../../xpcom/base/ClearOnShutdown.o collect2: ld returned 1 exit status gmake[1]: * [XUL] Error 1

This is very bad.

Comment #1

Posted on Mar 23, 2014 by Massive Rhino

This didn't happen in 26, and ICU is known to bloat the build substantially ( https://bugzilla.mozilla.org/show_bug.cgi?id=853301 ). Let's build without it.

Comment #2

Posted on Mar 23, 2014 by Massive Rhino

OK, that links. So that buys us some time until --with-intl-api becomes mandatory (it isn't yet because Android and B2G don't support it either).

Comment #3

Posted on Mar 24, 2014 by Happy Rabbit

Release builds should be smaller, shouldn't they? Even if not one can still dynamically link against ICU.

Comment #4

Posted on Mar 25, 2014 by Massive Rhino

They should be, but I haven't tried, and I still need a debug build.

Mozilla builds ICU static. They are working on something to spin out the .dat file which right now is linked in too, so that could help (right now the Mac version has two copies, one for x86 and one for x64, which bloats XUL further).

Comment #5

Posted on Mar 25, 2014 by Massive Rhino

And it looks like we're not the only one to hit this with ICU: http://tug.org/pipermail/tlbuild/2012q1/002109.html

Comment #6

Posted on Mar 25, 2014 by Massive Rhino

Well, I take that back. They do still support building a shared ICU; I thought it had bitrotted: https://bugzilla.mozilla.org/show_bug.cgi?id=926980

Comment #7

Posted on Jul 5, 2014 by Massive Rhino

We want to fix this eventually, but this is no longer critical.

Comment #8

Posted on Jun 15, 2015 by Massive Rhino

(No comment was entered for this change.)

Comment #9

Posted on Jul 2, 2015 by Massive Rhino

(No comment was entered for this change.)

Comment #10

Posted on Jul 29, 2015 by Massive Rhino

(No comment was entered for this change.)

Comment #11

Posted on Jul 29, 2015 by Massive Rhino

Also added a stub normalize() for some addons that require it.

Status: Accepted

Labels:
Type-Defect Priority-Medium