From 7a7e1b0c5cd2e7ea42c83e339e465b4ad32cfd50 Mon Sep 17 00:00:00 2001 From: Pablo Ulloa Date: Sun, 24 Sep 2023 20:39:19 +0000 Subject: [PATCH 1/4] [java] fixed feature test scenario check --- test/java/install_from_non_default_distro.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/java/install_from_non_default_distro.sh b/test/java/install_from_non_default_distro.sh index d4f53818e..021a72a21 100644 --- a/test/java/install_from_non_default_distro.sh +++ b/test/java/install_from_non_default_distro.sh @@ -5,7 +5,7 @@ set -e # Optional: Import test library source dev-container-features-test-lib -check "java version openjdk 11 installed" grep "openjdk 11." <(java --version) +check "java version openjdk 19 installed" grep "openjdk 19." <(java --version) # Report result reportResults From d53f4f1c1d9cc34e537d32188d9ca5341467918a Mon Sep 17 00:00:00 2001 From: Pablo Ulloa Date: Sun, 24 Sep 2023 20:40:24 +0000 Subject: [PATCH 2/4] [java] support for non conentional version --- src/java/install.sh | 4 ++-- test/java/install_non_conventional_version.sh | 11 +++++++++++ test/java/scenarios.json | 9 +++++++++ 3 files changed, 22 insertions(+), 2 deletions(-) create mode 100644 test/java/install_non_conventional_version.sh diff --git a/src/java/install.sh b/src/java/install.sh index ea43dd3e9..7021e05bb 100644 --- a/src/java/install.sh +++ b/src/java/install.sh @@ -111,13 +111,13 @@ sdk_install() { elif echo "${requested_version}" | grep -oE "${full_version_check}" > /dev/null 2>&1; then echo "${requested_version}" else - local regex="${prefix}\\K[0-9]+\\.[0-9]+\\.[0-9]+${suffix}" + local regex="${prefix}\\K[0-9]+\\.?[0-9]*\\.?[0-9]*${suffix}" local version_list=$(su ${USERNAME} -c ". \${SDKMAN_DIR}/bin/sdkman-init.sh && sdk list ${install_type} 2>&1 | grep -oP \"${regex}\" | tr -d ' ' | sort -rV") if [ "${requested_version}" = "latest" ] || [ "${requested_version}" = "current" ]; then requested_version="$(echo "${version_list}" | head -n 1)" else set +e - requested_version="$(echo "${version_list}" | grep -E -m 1 "^${requested_version//./\\.}([\\.\\s]|$)")" + requested_version="$(echo "${version_list}" | grep -E -m 1 "^${requested_version//./\\.}([\\.\\s]|-\s*|$)")" set -e fi if [ -z "${requested_version}" ] || ! echo "${version_list}" | grep "^${requested_version//./\\.}$" > /dev/null 2>&1; then diff --git a/test/java/install_non_conventional_version.sh b/test/java/install_non_conventional_version.sh new file mode 100644 index 000000000..3ab648cab --- /dev/null +++ b/test/java/install_non_conventional_version.sh @@ -0,0 +1,11 @@ +#!/bin/bash + +set -e + +# Optional: Import test library +source dev-container-features-test-lib + +check "java version openjdk 21 installed" grep "openjdk 21." <(java --version) + +# Report result +reportResults diff --git a/test/java/scenarios.json b/test/java/scenarios.json index 11fb7ba33..5d7e684ba 100644 --- a/test/java/scenarios.json +++ b/test/java/scenarios.json @@ -56,5 +56,14 @@ "mavenVersion": "3.6.3" } } + }, + "install_non_conventional_version": { + "image": "ubuntu:focal", + "features":{ + "java": { + "version": "21", + "jdkDistro": "graalce" + } + } } } From bea9fc2a69f2aeafd34ead67d138d29f0fe2a901 Mon Sep 17 00:00:00 2001 From: Pablo Ulloa Date: Tue, 26 Sep 2023 22:44:32 -0300 Subject: [PATCH 3/4] Update src/java/install.sh to not include whitespaces Co-authored-by: Samruddhi Khandale --- src/java/install.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/java/install.sh b/src/java/install.sh index 7021e05bb..2decc31b5 100644 --- a/src/java/install.sh +++ b/src/java/install.sh @@ -117,7 +117,7 @@ sdk_install() { requested_version="$(echo "${version_list}" | head -n 1)" else set +e - requested_version="$(echo "${version_list}" | grep -E -m 1 "^${requested_version//./\\.}([\\.\\s]|-\s*|$)")" + requested_version="$(echo "${version_list}" | grep -E -m 1 "^${requested_version//./\\.}([\\.\\s]|-|$)")" set -e fi if [ -z "${requested_version}" ] || ! echo "${version_list}" | grep "^${requested_version//./\\.}$" > /dev/null 2>&1; then From 5ec137e37defb29fb33a78478196c8e09a201574 Mon Sep 17 00:00:00 2001 From: Pablo Ulloa Date: Wed, 27 Sep 2023 01:45:10 +0000 Subject: [PATCH 4/4] bumped patch version --- src/java/devcontainer-feature.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/java/devcontainer-feature.json b/src/java/devcontainer-feature.json index 7bdec6ab0..d4aede873 100644 --- a/src/java/devcontainer-feature.json +++ b/src/java/devcontainer-feature.json @@ -1,6 +1,6 @@ { "id": "java", - "version": "1.2.1", + "version": "1.2.2", "name": "Java (via SDKMAN!)", "documentationURL": "https://github.com/devcontainers/features/tree/main/src/java", "description": "Installs Java, SDKMAN! (if not installed), and needed dependencies.",