[GR-65416] Use -R:+EnableSignalHandling -R:+InstallSegfaultHandler instead of forcing them in the thin launchers

Author: eregon

sdk/mx.sdk/mx_sdk_vm_impl.py

@@ -1417,6 +1417,8 @@ def contents(self):
 
             if isinstance(image_config, mx_sdk.LauncherConfig) or (isinstance(image_config, mx_sdk.LanguageLibraryConfig) and image_config.launchers):
                 build_args += [
+                    '-R:+EnableSignalHandling',
+                    '-R:+InstallSegfaultHandler',
                     '--enable-monitoring=jvmstat,heapdump,jfr,threaddump',
                 ] + svm_experimental_options([
                     '-H:+InstallExitHandlers',

sdk/mx.sdk/mx_sdk_vm_ng.py

@@ -376,7 +376,7 @@ class LanguageLibraryProject(NativeImageLibraryProject):
     def get_build_args(self):
         build_args = super().get_build_args()[:]
 
-        # Signals flags, the first 2 are also set in AbstractLanguageLauncher but better to be explicit
+        # Signals flags
         build_args += [
             '-R:+EnableSignalHandling',
             '-R:+InstallSegfaultHandler',

sdk/src/org.graalvm.launcher/src/org/graalvm/launcher/AbstractLanguageLauncher.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2017, 2024, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2017, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -48,7 +48,6 @@
 import java.util.List;
 import java.util.Map;
 
-import org.graalvm.nativeimage.RuntimeOptions;
 import org.graalvm.polyglot.Context;
 import org.graalvm.polyglot.Engine;
 import org.graalvm.polyglot.Language;
@@ -150,12 +149,6 @@ protected final void launch(String[] args) {
      * @throws Exception if no launcher constructor has been set.
      */
     public static void runLauncher(byte[][] optionVarsArgs, byte[][] args, int argc, long argv, boolean relaunch) throws Exception {
-        if (isAOT()) {
-            // enable signal handling for the launcher
-            RuntimeOptions.set("EnableSignalHandling", true);
-            RuntimeOptions.set("InstallSegfaultHandler", true);
-        }
-
         if (LAUNCHER_CTOR == null) {
             throw new Exception("Launcher constructor has not been set.");
         }