Partial fix for #595 - Add explicit check for java binary

Signed-off-by: Rob Stryker <rob@oxbeef.net>
redhat-developer · Apr 1, 2024 · bb3d7f3 · bb3d7f3
1 parent 33105d3
commit bb3d7f3
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/impl/requirements.ts b/src/impl/requirements.ts
@@ -101,6 +101,9 @@ async function checkJavaRuntime(): Promise<string | RspRequirementsRejection> {
         if (!pathExists.sync(javaHome)) {
             return getRejectionWithDownloadUrl(`${source} points to a missing folder`);
         }
+        if (!pathExists.sync(path.resolve(javaHome, 'bin', JAVA_FILENAME))) {
+            return getRejectionWithDownloadUrl(`${source} does not point to a JDK. The '${JAVA_FILENAME}' command is missing.`);
+        }
         if (!pathExists.sync(path.resolve(javaHome, 'bin', JAVAC_FILENAME))) {
             return getRejectionWithDownloadUrl(`${source} does not point to a JDK. The '${JAVAC_FILENAME}' command is missing.`);
         }