Author: ken
Date: Tue Dec  3 04:20:30 2019
New Revision: 22415

Log:
Fix qtwebengine for ICU-65.

Modified:
   trunk/BOOK/introduction/welcome/changelog.xml
   trunk/BOOK/x/lib/qtwebengine.xml

Modified: trunk/BOOK/introduction/welcome/changelog.xml
==============================================================================
--- trunk/BOOK/introduction/welcome/changelog.xml       Mon Dec  2 23:57:44 
2019        (r22414)
+++ trunk/BOOK/introduction/welcome/changelog.xml       Tue Dec  3 04:20:30 
2019        (r22415)
@@ -45,6 +45,10 @@
       <para>December 3rd, 2019</para>
       <itemizedlist>
         <listitem>
+          <para>[ken] - Fix build breakage from ICU-65.1 in qtwebengine-5.13.2,
+          again caused by a U16NEXT() without a terminating semicolon.</para>
+        </listitem>
+        <listitem>
           <para>[ken] - Do not use system ICU for libreoffice, 65.1 broke
           one of the packages which it ships with (libfreehand). Fixes
           <ulink url="&blfs-ticket-root;12841">#12841</ulink>.</para>

Modified: trunk/BOOK/x/lib/qtwebengine.xml
==============================================================================
--- trunk/BOOK/x/lib/qtwebengine.xml    Mon Dec  2 23:57:44 2019        (r22414)
+++ trunk/BOOK/x/lib/qtwebengine.xml    Tue Dec  3 04:20:30 2019        (r22415)
@@ -186,6 +186,19 @@
     -i src/3rdparty/chromium/media/media_options.gni</userinput></screen>
 
     <para>
+      Now add a terminating semicolon to a macro to let it compile with
+      <xref linkend="icu"/>:
+    </para>
+
+<!-- editors: this line number is specific to 5.13.2, later versions might
+have already been fixed, or more likely the relevant line is no longer 5713.
+There are two lines which invoke U16_NOTE(), the first already has a ';'
+followed by a comment, the second lacks the ';' -->
+
+<screen><userinput>sed -i '5713s/.*/&amp;;/' \
+  
src/3rdparty/chromium/third_party/blink/renderer/core/dom/document.cc</userinput></screen>
+
+    <para>
       If an older version of the package's main library has been installed,
       when the package is built separately it will link to that in preference
       to its own not-yet-installed version, and fail because of missing 
symbols.
-- 
http://lists.linuxfromscratch.org/listinfo/blfs-book
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to