Force early init of JVM process logic so CHLD traps work.

Fixes #3283.
jruby · Aug 25, 2015 · 627d84c · 627d84c
1 parent a558250
commit 627d84c
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/core/src/main/java/org/jruby/RubySignal.java b/core/src/main/java/org/jruby/RubySignal.java
@@ -31,6 +31,7 @@
 
 import org.jruby.anno.JRubyMethod;
 import org.jruby.anno.JRubyModule;
+import org.jruby.platform.Platform;
 import org.jruby.runtime.ThreadContext;
 import org.jruby.runtime.builtin.IRubyObject;
 import org.jruby.util.SignalFacade;
@@ -52,6 +53,17 @@ private final static SignalFacade getSignalFacade() {
     }
 
     public static void createSignal(Ruby runtime) {
+        // We force java.lang.Process et al to load so that JVM's CHLD handler can be
+        // overwritten by users (jruby/jruby#3283)
+        if (!Platform.IS_WINDOWS) {
+            try {
+                Class.forName("java.lang.Process");
+                Class.forName("java.lang.UNIXProcess");
+            } catch (Throwable t) {
+                // if we can't access Process, other things will fail anyway; ignore for now
+            }
+        }
+
         RubyModule mSignal = runtime.defineModule("Signal");
 
         mSignal.defineAnnotatedMethods(RubySignal.class);