Skip to content
This repository was archived by the owner on Feb 4, 2023. It is now read-only.

Commit 0e6c117

Browse files
committed
[refactoring] small changes
1 parent cb07a77 commit 0e6c117

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

src/universalJavaApplicationStub

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -495,10 +495,12 @@ function does_java_version_satisfy_requirement() {
495495
# find installed Java versions
496496
############################################
497497

498+
# default Apple JRE plugin path
498499
apple_jre_plugin="/Library/Java/Home/bin/java"
499-
apple_jre_version=$(extract_java_major_version $(get_java_version_from_cmd "${apple_jre_plugin}"))
500+
apple_jre_version=$(get_java_version_from_cmd "${apple_jre_plugin}")
501+
# default Oracle JRE plugin path
500502
oracle_jre_plugin="/Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java"
501-
oracle_jre_version=$(extract_java_major_version $(get_java_version_from_cmd "${oracle_jre_plugin}"))
503+
oracle_jre_version=$(get_java_version_from_cmd "${oracle_jre_plugin}")
502504

503505
# first check system variable "$JAVA_HOME"
504506
if [ -n "$JAVA_HOME" ] ; then
@@ -557,7 +559,7 @@ elif [ ! -z ${JVMVersion} ] ; then
557559

558560
if [ -z "${JAVACMD}" ] ; then
559561
# display human readable java version (#28)
560-
java_version_hr=$(echo ${JVMVersion} | sed -E 's/^1\.([0-9+*]+)$/ \1/g' | sed "s/+/ ${MSG_JAVA_VERSION_OR_LATER}/" | sed "s/*/ ${MSG_JAVA_VERSION_LATEST}/")
562+
java_version_hr=$(echo ${JVMVersion} | sed -E 's/^1\.([0-9+*]+)$/ \1/g' | sed "s/+/ ${MSG_JAVA_VERSION_OR_LATER}/;s/*/ ${MSG_JAVA_VERSION_LATEST}/")
561563
MSG_NO_SUITABLE_JAVA_EXPANDED=$(printf "${MSG_NO_SUITABLE_JAVA}" "${java_version_hr}")
562564
# log exit cause
563565
stub_logger "[EXIT 3] ${MSG_NO_SUITABLE_JAVA_EXPANDED}"

0 commit comments

Comments
 (0)