From 1d5a3921cec923cca7f3f3d7fdd1c41fbab9f814 Mon Sep 17 00:00:00 2001 From: Charles Oliver Nutter Date: Wed, 1 May 2024 16:26:34 -0500 Subject: [PATCH] Load subspawn builtins by default on non-Windows The logic in subspawn is much more complete and passes more specs, so this is a first step toward using it by default on all platforms where it works. --- core/src/main/java/org/jruby/Ruby.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/core/src/main/java/org/jruby/Ruby.java b/core/src/main/java/org/jruby/Ruby.java index b59801dd4c7..39eda6c42ee 100644 --- a/core/src/main/java/org/jruby/Ruby.java +++ b/core/src/main/java/org/jruby/Ruby.java @@ -544,12 +544,8 @@ private Ruby(RubyInstanceConfig config) { // Done booting JRuby runtime runtimeIsBooted = true; - if ("true".equals(System.getenv("USE_SUBSPAWN"))) { - if (Platform.IS_WINDOWS) { - LOG.warn("env USE_SUBSPAWN=true is unsupported on Windows at this time"); - } else { - getLoadService().require("subspawn/replace-builtin"); - } + if (posix.isNative() && !Platform.IS_WINDOWS) { + getLoadService().require("subspawn/replace-builtin"); } }