config_host.mk.in                         |    1 -
 configure.ac                              |   10 ----------
 external/hsqldb/ExternalProject_hsqldb.mk |    2 --
 3 files changed, 13 deletions(-)

New commits:
commit 35d9ec3e596bce2ea35ed3b6418e0ba672a61443
Author:     Stephan Bergmann <stephan.bergm...@allotropia.de>
AuthorDate: Mon Mar 3 08:43:21 2025 +0100
Commit:     Stephan Bergmann <stephan.bergm...@allotropia.de>
CommitDate: Mon Mar 3 10:27:06 2025 +0100

    Remove the JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD workaround
    
    ...that had been introduced with 19de5d97efbb80e9767bc6b873bcc2c263a721de 
"Allow
    building against JDK 18" to work around odd failures of 
ExternalProject_hsqldb
    with whatever versions of JDK 18 and Ant I had been using at that time.
    
    However, with java-latest-openjdk-24.0.0.0.36-1.rolling.fc41.x86_64 that
    workaround now started to make ExternalProject_hsqldb fail for me with
    
    > ERROR: JVMTI: 112(JVMTI_ERROR_WRONG_PHASE): Cannot get thread info
    > ERROR: JVMTI: 112(JVMTI_ERROR_WRONG_PHASE): jvmtiEnv::GetLoadedClasses()
    > Error occurred during initialization of VM
    > java.lang.Error: A command line option has attempted to allow or enable 
the Security Manager. Enabling a Security Manager is not supported.
    >       at java.lang.System.initPhase3(java.base@24/System.java:1947)
    
    apparently because of <https://openjdk.org/projects/jdk/24/>'s
    <https://openjdk.org/jeps/486> "JEP 486: Permanently Disable the Security
    Manager".
    
    But with the ant-1.10.14-13.fc41.noarch I'm using now, neither
    java-21-openjdk-21.0.6.0.7-6.fc41 (I no longer have the 
problematic-back-then
    JDK 18) nor java-latest-openjdk-24.0.0.0.36-1.rolling.fc41.x86_64 appear to 
need
    that workaround anymore anyway (presumably because "an updated version of 
Apache
    Ant [has] become available", as discussed in the commit message of that
    19de5d97efbb80e9767bc6b873bcc2c263a721de).
    
    Change-Id: Id13b57559df486012c888247c96323716168c97d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/182419
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de>

diff --git a/config_host.mk.in b/config_host.mk.in
index 2ab62f748bd6..deafc48737d3 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -382,7 +382,6 @@ export JAVA_TARGET_VER=@JAVA_TARGET_VER@
 export JAWTLIB=@JAWTLIB@
 export JDK=@JDK@
 export JDK_FOR_BUILD=@JDK_FOR_BUILD@
-export 
JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD=@JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD@
 export JFREEREPORT_JAR=@JFREEREPORT_JAR@
 export JITC_PROCESSOR_TYPE=@JITC_PROCESSOR_TYPE@
 JOM=@JOM@
diff --git a/configure.ac b/configure.ac
index b4c4cfecf2e2..37dff9cb0d54 100644
--- a/configure.ac
+++ b/configure.ac
@@ -6236,7 +6236,6 @@ if test "$cross_compiling" = "yes"; then
         JAVA_HOME
         JAVAIFLAGS
         JDK
-        JDK_SECURITYMANAGER_DISALLOWED
         LIBO_BIN_FOLDER
         LIBO_LIB_FOLDER
         LIBO_URE_LIB_FOLDER
@@ -9081,7 +9080,6 @@ dnl 
===================================================================
 
 # Whether all the complexity here actually is needed any more or not, no idea.
 
-JDK_SECURITYMANAGER_DISALLOWED=
 MODULAR_JAVA=
 if test "$ENABLE_JAVA" != "" -a "$cross_compiling" != "yes"; then
     _gij_longver=0
@@ -9106,12 +9104,6 @@ if test "$ENABLE_JAVA" != "" -a "$cross_compiling" != 
"yes"; then
             if test "$_jdk_ver" -lt 10800; then
                 AC_MSG_ERROR([JDK is too old, you need at least 8 ($_jdk_ver < 
10800)])
             fi
-            dnl TODO: Presumably, the Security Manager will not merely be 
disallowed, but be
-            dnl completely removed in some Java version > 18 (see
-            dnl <https://openjdk.java.net/jeps/411> "Deprecate the Security 
Manager for Removal"):
-            if test "$_jdk_ver" -ge 180000; then
-                JDK_SECURITYMANAGER_DISALLOWED=TRUE
-            fi
 
             JAVA_HOME=`echo $JAVAINTERPRETER | $SED -n "s,//*bin//*java,,p"`
             if test "$_os" = "WINNT"; then
@@ -9421,7 +9413,6 @@ if test "$ENABLE_JAVA" != "" -a "$cross_compiling" != 
"yes"; then
     JAVA_HOME_FOR_BUILD=$JAVA_HOME
     JAVAIFLAGS_FOR_BUILD=$JAVAIFLAGS
     JDK_FOR_BUILD=$JDK
-    JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD=$JDK_SECURITYMANAGER_DISALLOWED
 fi
 
 AC_SUBST(JAVACFLAGS)
@@ -9433,7 +9424,6 @@ AC_SUBST(JAVA_HOME)
 AC_SUBST(JAVA_HOME_FOR_BUILD)
 AC_SUBST(JDK)
 AC_SUBST(JDK_FOR_BUILD)
-AC_SUBST(JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD)
 AC_SUBST(JAVA_SOURCE_VER)
 AC_SUBST(JAVA_TARGET_VER)
 AC_SUBST(MODULAR_JAVA)
diff --git a/external/hsqldb/ExternalProject_hsqldb.mk 
b/external/hsqldb/ExternalProject_hsqldb.mk
index 4e5c750c7f3c..a3dfcade6b87 100644
--- a/external/hsqldb/ExternalProject_hsqldb.mk
+++ b/external/hsqldb/ExternalProject_hsqldb.mk
@@ -22,8 +22,6 @@ $(call gb_ExternalProject_get_state_target,hsqldb,build) :
        $(call gb_Trace_StartRange,hsqldb,EXTERNAL)
        $(call gb_ExternalProject_run,build,\
                unset MSYS_NO_PATHCONV && JAVA_HOME=$(JAVA_HOME_FOR_BUILD) \
-               $(if $(JDK_SECURITYMANAGER_DISALLOWED_FOR_BUILD), \
-                   ANT_OPTS="$$ANT_OPTS -Djava.security.manager=allow") \
                $(ICECREAM_RUN) "$(ANT)" \
                        $(if $(verbose),-v,-q) \
                        -f build/build.xml \

Reply via email to