Changeset ac465d6c for general/genlib/js102.xml

Timestamp:

07/23/2023 12:43:34 AM (11 months ago)

Author:

Xi Ruoyao <xry111@…>

Branches:

12.0, 12.1, ken/TL2024, ken/tuningfonts, lazarus, plabs/newcss, python3.11, rahul/power-profiles-daemon, renodr/vulkan-addition, trunk, xry111/llvm18

Children:

1f2235c

Parents:

aa1bc104

git-author:

Xi Ruoyao <xry111@…> (07/23/2023 12:42:32 AM)

git-committer:

Xi Ruoyao <xry111@…> (07/23/2023 12:43:34 AM)

Message:

js102: Remove commented out stale work around and add <important> for GCC 32-bit issue

Let's stop puzzling the editors.

File:

• general/genlib/js102.xml (modified) (2 diffs)

general/genlib/js102.xml

@@ -107 +107 @@
     </para>
 
+    <important>
+      <para>
+        If you are building this package on a 32-bit system, and Clang
+        is not installed or you're overriding the default compiler choice
+        with the environment variable <envar>CXX</envar>, please read the
+        Command Explanations section first.
+      </para>
+    </important>
+
     <!-- It seems nasm is only used for aarch64-win64. -->
     <!--bridgehead renderas="sect4">Optional</bridgehead>
@@ -147 +156 @@
 
     </note>
-<!-- not needed when using clang.  Upstream bug at
-    https://bugzilla.mozilla.org/show_bug.cgi?id=1729459.
-    07-21-2023: Issue has resurfaced again when using the default build
-                instructions for JS (which uses GCC). The problem is not present
-                in Firefox 115 though (which works OK on i686). -renodr
-
-
-    <para>
-      If building on a 32 bit machine, the following sed works around an issue
-      with an incompatible <quote>double</quote> type in one header file:
-    </para>
-
-<screen><userinput>case "$(uname -m)" in
-    i?86) sed -e 's/typedef double/typedef long double/' \
-              -i modules/fdlibm/src/math_private.h ;;
-esac</userinput></screen>
--->
 
     <para>