Test successful JRuby start without using launcher

headius · eregon · commit 6c79f721fa26 · 2025-03-13T20:32:49.000+01:00
Native launcher appears to hang on GHA when the JDK version is too
old for JRuby 10. This bypasses it and should be a bit lighter. It
will also avoid any variability in the launcher used, since Unixes
and Windows already use different executables.
diff --git a/common.js b/common.js
@@ -405,11 +405,11 @@ export function setupPath(newPathEntries) {
   return msys2Type
 }
 
-export async function setupJavaHome() {
+export async function setupJavaHome(rubyPrefix) {
   await measure("Modifying JAVA_HOME for JRuby", async () => {
     console.log("attempting to run with existing JAVA_HOME")
 
-    let ret = await exec.exec('ruby', ['--version'], {ignoreReturnCode: true})
+    let ret = await exec.exec('java', ['-jar', path.join(rubyPrefix, 'lib/jruby.jar'), '--version'], {ignoreReturnCode: true})
 
     if (ret === 0) {
       console.log("JRuby successfully starts, using existing JAVA_HOME")
diff --git a/dist/index.js b/dist/index.js
diff --git a/ruby-builder.js b/ruby-builder.js
@@ -56,7 +56,7 @@ export async function install(platform, engine, version) {
 
   // Ensure JRuby has minimum Java version to run
   if (engine === "jruby") {
-    await common.setupJavaHome()
+    await common.setupJavaHome(rubyPrefix)
   }
 
   return rubyPrefix
