Control: tags -1 + patch
On Sat, 23 Jun 2018 at 16:33:40 +0100, Simon McVittie wrote:
> On Sat, 23 Jun 2018 at 12:52:01 +0300, Adrian Bunk wrote:
> > make[3]: *** No rule to make target 'ParagraphLayout.ao', needed by
> > '../lib/libiculx.a'. Stop.
>
> It looks as though Ubuntu might have a fix for this
> ...
> + * Don't build-depend on libicu-dev
This didn't work for me: even with a Build-Conflicts, I got the same
failure. However, a better solution seems to be to pass --disable-layoutex
to the bundled icu's configure script, as in the attached. That makes
some sense to me at least: the bundled icu's configure script has an
"automagic" dependency on the icu-le-hb layout engine library, which until
recently we didn't have. However, the bundled icu is incomplete/minimal
and doesn't actually have the source code for the optional paragraph
layout feature that requires icu-le-hb. We can resolve that by turning
the feature off, which isn't a regression because the versions of mozjs52
that compiled successfully didn't have it either.
Having fixed that, I get a different FTBFS: the mozjs command-line entry
point fails to start up. I'll open a separate bug to track that. I can
work around it, but I'm reasonably sure my workaround isn't the proper
solution.
smcv
From: Mike Hommey <[email protected]>
Date: Sat, 19 May 2018 11:23:18 +0900
Subject: Add --disable-layoutex when running ICU configure. r=froydnj,
a=RyanVM
In some recent ICU update, --enable-layoutex changed from defaulting
to the "same-as-layout" to "yes", so --disable-layout doesn't imply
--disable-layoutex anymore, but we removed the layoutex files.
Applied-upstream: https://hg.mozilla.org/mozilla-unified/rev/31d4880fde31
---
intl/icu_sources_data.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/intl/icu_sources_data.py b/intl/icu_sources_data.py
index 7d2d983..fc95d6a 100644
--- a/intl/icu_sources_data.py
+++ b/intl/icu_sources_data.py
@@ -122,6 +122,7 @@ def update_data_file(topsrcdir):
'--disable-extras',
'--disable-icuio',
'--disable-layout',
+ '--disable-layoutex',
'--disable-tests',
'--disable-samples',
'--disable-strict'],
