[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1079438: marked as done (libreoffice: FTBFS on armel, mips64el, loong64, powerpc: libjvm.so location change in openjdk-21 Zero VM)

Your message dated Sun, 25 Aug 2024 11:45:52 +0000
with message-id <E1siBgu-00HULf-FH@fasolo.debian.org>
and subject line Bug#1079438: fixed in libreoffice 4:24.2.5-4
has caused the Debian Bug report #1079438,
regarding libreoffice: FTBFS on armel, mips64el, loong64, powerpc: libjvm.so location change in openjdk-21 Zero VM
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1079438: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1079438
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Source: libreoffice
Severity: serious
Tags: ftbfs patch upstream
Justification: fails to build from source (but built successfully in the past)

Dear Maintainer,

Recently, the default-jdk was switched to openjdk-21 in unstable.
However in OpenJDK 21+ Zero VM builds, the location of libjvm.so has
changed from $JAVA_HOME/lib/server to $JAVA_HOME/lib/zero, making the
LO plugin loader unable to locate the JVM thus breaking the build.

Example build log message:

> exception occurred: Could not create Java implementation loader at
> ./stoc/source/javaloader/javaloader.cxx:551

Build logs of 24.2.5-3 that show the failure:

- https://buildd.debian.org/status/fetch.php?pkg=libreoffice&arch=mips64el&ver=4%3A24.2.5-3&stamp=1723467805&raw=0
- https://buildd.debian.org/status/fetch.php?pkg=libreoffice&arch=loong64&ver=4%3A24.2.5-3&stamp=1724299296&raw=0
- https://buildd.debian.org/status/fetch.php?pkg=libreoffice&arch=powerpc&ver=4%3A24.2.5-3&stamp=1723338137&raw=0

I have created a fix upstream at
https://gerrit.libreoffice.org/c/core/+/172248 which I tested locally to
fix the Debian build. Before the fix is integrated upstream, the patch
could be individually applied to allow the current version(s) to build.

Thanks!

-- System Information:
Debian Release: trixie/sid
  APT prefers unreleased
  APT policy: (500, 'unreleased'), (500, 'unstable')
Architecture: loong64 (loongarch64)

Kernel: Linux 6.9.9-gentoo-dist (SMP w/8 CPU threads; PREEMPT)
Kernel taint flags: TAINT_OOT_MODULE
Locale: LANG=C.UTF-8, LC_CTYPE=C.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)

>From 85aecfe4fa9b28baf4b4b44b80f03d14f91c5eb2 Mon Sep 17 00:00:00 2001
From: WANG Xuerui <xen0n@gentoo.org>
Date: Thu, 22 Aug 2024 14:22:49 +0800
Subject: [PATCH] sunjre: check all four possible OpenJDK variants for
 libjvm.so

Since OpenJDK 21, the "minimal" and "zero" variants of JVM no longer
reside in the "server" directory in $JAVA_HOME/lib, making the loader
unable to find libjvm.so. This regresses LO builds on platforms with
only those variants of JVM (i.e. without a HotSpot port).

Example error message is like this:

> exception occurred: Could not create Java implementation loader at
> ./stoc/source/javaloader/javaloader.cxx:551

Fix it by probing all four possible variant directories for libjvm.so.
This is tested on Debian unstable (loong64 port) to fix the packaging.

Link: https://bugs.openjdk.org/browse/JDK-8273494
Link: https://github.com/openjdk/jdk/pull/5440
Link: https://github.com/openjdk/jdk/commit/8fbcc8239a3fc04e56ebbd287c7bb5db731977b7
Change-Id: I3545f93054d7a3af0181e4c739c82efaaf8aeaed
---
 jvmfwk/plugins/sunmajor/pluginlib/sunjre.cxx | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/jvmfwk/plugins/sunmajor/pluginlib/sunjre.cxx b/jvmfwk/plugins/sunmajor/pluginlib/sunjre.cxx
index a0f8cbee1f3e..926a0e469935 100644
--- a/jvmfwk/plugins/sunmajor/pluginlib/sunjre.cxx
+++ b/jvmfwk/plugins/sunmajor/pluginlib/sunjre.cxx
@@ -63,10 +63,15 @@ char const* const* SunInfo::getRuntimePaths(int * size)
         "/lib/jli/libjli.dylib"
 #elif defined UNX
         "/lib/" JFW_PLUGIN_ARCH "/client/libjvm.so",
+        "/lib/" JFW_PLUGIN_ARCH "/minimal/libjvm.so",
         "/lib/" JFW_PLUGIN_ARCH "/server/libjvm.so",
+        "/lib/" JFW_PLUGIN_ARCH "/zero/libjvm.so",
         "/lib/" JFW_PLUGIN_ARCH "/classic/libjvm.so",
         "/lib/" JFW_PLUGIN_ARCH "/jrockit/libjvm.so",
-        "/lib/server/libjvm.so"
+        "/lib/client/libjvm.so",
+        "/lib/minimal/libjvm.so",
+        "/lib/server/libjvm.so",
+        "/lib/zero/libjvm.so"
 #endif
     };
     *size = SAL_N_ELEMENTS(ar);
@@ -78,7 +83,9 @@ char const* const* SunInfo::getLibraryPaths(int* size)
 #if defined UNX && !defined MACOSX
     static char const * ar[] = {
         "/lib/" JFW_PLUGIN_ARCH "/client",
+        "/lib/" JFW_PLUGIN_ARCH "/minimal",
         "/lib/" JFW_PLUGIN_ARCH "/server",
+        "/lib/" JFW_PLUGIN_ARCH "/zero",
         "/lib/" JFW_PLUGIN_ARCH "/native_threads",
         ("/lib/" JFW_PLUGIN_ARCH)
     };
-- 
2.45.2

--- End Message ---
--- Begin Message --- 
Source: libreoffice
Source-Version: 4:24.2.5-4
Done: Rene Engelhard <rene@debian.org>

We believe that the bug you reported is fixed in the latest version of
libreoffice, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1079438@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Rene Engelhard <rene@debian.org> (supplier of updated libreoffice package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Sun, 25 Aug 2024 13:06:48 +0200
Source: libreoffice
Architecture: source
Version: 4:24.2.5-4
Distribution: unstable
Urgency: medium
Maintainer: Debian LibreOffice Maintainers <debian-openoffice@lists.debian.org>
Changed-By: Rene Engelhard <rene@debian.org>
Closes: 1079438
Changes:
 libreoffice (4:24.2.5-4) unstable; urgency=medium
 .
   * debian/patches/jdk-minimal-and-zero-paths.diff: add new
     minimal/zero JDK paths as of OpenJDK 21, thanks
     WANG Xuerui <i.salsa@xen0n.name> (closes: #1079438).
     Keep Java disabled on armel though for consistency with armhf
 .
   * debian/patches/moreIconsDialog-accesses-internet.diff: fix logic
   * debian/tests/control.in: go sure and add needs-internet restriction here,
     too (it runs a additiondialog UI test)
Checksums-Sha1:
 b9942b0787ad6564741079296d26c556bdf39027 36993 libreoffice_24.2.5-4.dsc
 d0a0d7882355815dddac1a34b5464f5f63144976 24901024 libreoffice_24.2.5-4.debian.tar.xz
 c816345099538174c36c96090e30113b7326c3d7 25915 libreoffice_24.2.5-4_source.buildinfo
Checksums-Sha256:
 a82d85ef11435e55010e7c67df94a44def8df6f7ade58938773c119973198a0a 36993 libreoffice_24.2.5-4.dsc
 614a1bbdb3c620cb0c4acfa0c46abdc4dc81845a194e2fce2d57e504f9821586 24901024 libreoffice_24.2.5-4.debian.tar.xz
 2b7f9d3652c414890fe3b346d8d38693ff9573810d35f364a5cdf3ac61769383 25915 libreoffice_24.2.5-4_source.buildinfo
Files:
 bcc9fd48b1cf31e6a20e0a4bcce50bc4 36993 editors optional libreoffice_24.2.5-4.dsc
 9317431b51b9d382220b93ee5d763b1b 24901024 editors optional libreoffice_24.2.5-4.debian.tar.xz
 c871870a9121a3468a325909587066df 25915 editors optional libreoffice_24.2.5-4_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJEBAEBCgAuFiEE4S3qRnUGcM+pYIAdCqBFcdA+PnAFAmbLEZwQHHJlbmVAZGVi
aWFuLm9yZwAKCRAKoEVx0D4+cDr+D/9NXIS65MHVmJXSIH6D31Ep81ziVeQ1VNU/
/6qzTgBoVodl6jFwFbQK3reI9ADRl5MngczxLlMapF43W+BKLe/T7R/R+Voexcfn
/mC3G0AldJ93fW8hLZHlFMoRlI70Ft1DTJLQJkTMgDXahEcPmipMdTwtGVMrVQ5p
PwgMzp1lEOxYH1mqalpD45rwvOdinYLu8HmsoFdhchx5kYgY/HRwsqKiM0ltKN0F
Rd42mO+bemPH9tH83okE4osou+2Ej1eK69yjl15k3cyHqSBZSCHZE+43Kbd5DV1S
MpJnw8LeU3G476QZEV2eR8Xzfxz5tGfKg/MmpnHlPb/KGBx+G8QZgbknqJXSAcoF
An5pq1m225Fw2pssVZCrL4KTSabSJZnAVmFLGFybjMNKL3hMRPKmcKPxt0FTlwRA
YtCwa/tGWNKIGYSBRDQU4Fgn8bIawFYvFOwrMAtlwJFGEohWH/i3h2Ns7wS8N85M
sKnQBD2VlY6EGRX0O96fx+HtQ2JfI+SCkv4sB0PzHFoXzYCoA6dBUvNWRYphQJw5
Dk4xNYl7GSucU2v3R20DqgAYRdpdXGLlFwLW1mBGPOwjpj/Wi3EfgWgXnpzNwsaa
XZdIMoHSsYD4lUEqOlS1phEOoYEXcTVAsAwfrQEwFr84dvckgjhIer0B6UA/9LB4
Z7iMDpw5dQ==
=zbEc
-----END PGP SIGNATURE-----

Attachment: pgpu7XqB9SBTW.pgp
Description: PGP signature


--- End Message ---
Reply to: