maven and optional sdkman (#25)

* maven

* optional sdk man

.github/workflows/test-all.yaml

@@ -30,7 +30,7 @@ jobs:
             "ruby jekyll",        # Install 'ruby', then 'jekyll'
             "python jupyterlab",  # Install 'python', then 'jupyterlab'
             "kubectl-helm-minikube",
-            "maven",
+            "java maven",         # Install 'java', then 'maven'
             "node",
             "php",
             "powershell",

.github/workflows/test-pr.yaml

@@ -30,7 +30,7 @@ jobs:
           jekyll: ./**/jekyll/**
           'python jupyterlab': ./**/jupyterlab/**
           kubectl-helm-minikube: ./**/kubectl-helm-minikube/**
-          maven: ./**/maven/**
+          'java maven': ./**/maven/**
           node: ./**/node/**
           php: ./**/php/**
           powershell: ./**/powershell/**

src/gradle/feature.json

@@ -5,14 +5,22 @@
     "options": {
         "version": {
             "type": "string",
-            "proposals": ["latest", "none", "7", "6", "5"],
+            "proposals": [
+                "latest",
+                "none",
+                "7",
+                "6",
+                "5"
+            ],
             "default": "latest",
             "description": "Select or enter a Gradle version to install"
         }
     },
-    "extensions": ["vscjava.vscode-java-pack"],
+    "extensions": [
+        "vscjava.vscode-java-pack"
+    ],
     "containerEnv": {
-        "SDKMAN_DIR": "/usr/local/sdkman",
+        "SDKMAN_DIR": "${SDKMAN_DIR:-\"/usr/local/sdkman\"}",
         "PATH": "${SDKMAN_DIR}/bin:${SDKMAN_DIR}/candidates/gradle/current/bin:${PATH}"
     },
     "install": {

src/maven/feature.json

@@ -5,14 +5,22 @@
     "options": {
         "version": {
             "type": "string",
-            "proposals": ["latest", "none", "3.8", "3.6", "3.5"],
+            "proposals": [
+                "latest",
+                "none",
+                "3.8",
+                "3.6",
+                "3.5"
+            ],
             "default": "latest",
             "description": "Select or enter a Maven version to install"
         }
     },
-    "extensions": ["vscjava.vscode-java-pack"],
+    "extensions": [
+        "vscjava.vscode-java-pack"
+    ],
     "containerEnv": {
-        "SDKMAN_DIR": "/usr/local/sdkman",
+        "SDKMAN_DIR": "${SDKMAN_DIR:-\"/usr/local/sdkman\"}",
         "PATH": "${SDKMAN_DIR}/bin:${SDKMAN_DIR}/candidates/maven/current/bin:${PATH}"
     },
     "install": {

src/maven/install.sh

@@ -7,12 +7,13 @@
 # Docs: https://github.com/microsoft/vscode-dev-containers/blob/main/script-library/docs/maven.md
 # Maintainer: The VS Code and Codespaces Teams
 #
-# Syntax: ./maven-debian.sh [maven version] [SDKMAN_DIR] [non-root user] [Update rc files flag]
+# Syntax: ./maven-debian.sh [maven version] [non-root user] [Update rc files flag]
 
 MAVEN_VERSION=${1:-"latest"}
-export SDKMAN_DIR=${2:-"/usr/local/sdkman"}
-USERNAME=${3:-"automatic"}
-UPDATE_RC=${4:-"true"}
+USERNAME=${2:-"automatic"}
+UPDATE_RC=${3:-"true"}
+
+SDKMAN_DIR=${SDKMAN_DIR:-"/usr/local/sdkman"}
 
 set -e
 

test/maven/test.sh

@@ -6,7 +6,7 @@ set -e
 source dev-container-features-test-lib
 
 # Definition specific tests
-check "version" maven  --version
+check "version" mvn  --version
 
 # Report result
 reportResults
\ No newline at end of file