Merge pull request #449 from jboss-container-images/OPENJDK-2736-jlink-fail-early-jmods

OPENJDK-2736 and OPENJDK-2737: gate Jlink code, fail early if jmods unavailable

Author: jmtd

modules/jlink/artifacts/opt/jboss/container/java/jlink/preflight.sh

@@ -0,0 +1,14 @@
+jlink_preflight_check()
+{
+    # preflight check: do we have what we need?
+    if [ "$JAVA_VERSION" -lt 11 ]; then
+      echo "Jlink integration not available for JDK${JAVA_VERSION}!"
+      echo "Jlink integration is only supported for JDK versions 11 and newer."
+      exit 1
+    fi
+    if [ ! -d /usr/lib/jvm/java/jmods ]; then
+      echo "Jlink integration requires the jmods RPM to be installed in the builder image, e.g."
+      echo "        microdnf install -y java-${JAVA_VERSION}-openjdk-jmods"
+      exit 1
+    fi
+}

modules/s2i/bash/artifacts/usr/local/s2i/assemble

@@ -16,22 +16,28 @@ maven_s2i_build
 
 # run the pathfinder scripts to define JAVA_APP_JAR and JAVA_LIB_DIR
 source "${JBOSS_CONTAINER_UTIL_PATHFINDER_MODULE}/pathfinder.sh"
-echo "Setting up java app and lib variables"
 setup_java_app_and_lib
 
 # include our jlink scripts
-source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkdeps.sh"
-echo "Invoking mkdeps"
-generate_deps
+if [ "$S2I_ENABLE_JLINK" = "true" ]; then
 
-source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkstrippeddeps.sh"
-echo "Stripping dependencies"
-mkstrippeddeps
+    source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/preflight.sh"
+    jlink_preflight_check
 
-source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/generatejdkdeps.sh"
-echo "Generating JDK dependencies"
-generatejdkdeps
+    source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkdeps.sh"
+    echo "Invoking mkdeps"
+    generate_deps
 
-source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkjreimage.sh"
-echo "Linking jre"
-generate_jre_image
+    source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkstrippeddeps.sh"
+    echo "Stripping dependencies"
+    mkstrippeddeps
+
+    source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/generatejdkdeps.sh"
+    echo "Generating JDK dependencies"
+    generatejdkdeps
+
+    source "${JBOSS_CONTAINER_JAVA_JLINK_MODULE}/mkjreimage.sh"
+    echo "Linking jre"
+    generate_jre_image
+
+fi