Fix: Maven wrapper script quoting bug prevents Java fallback detection

[coderabbitai]

Bug Description

In the Maven wrapper script (mvnw), there's a bug where single quotes around shell built-ins prevent Java detection from working properly.

The following code in mvnw:

JAVACMD="$(
  'set' +e
  'unset' -f command 2>/dev/null
  'command' -v java
)" || :
JAVACCMD="$(
  'set' +e
  'unset' -f command 2>/dev/null
  'command' -v javac
)" || :

Should be fixed by removing the single quotes:

JAVACMD="$(
  set +e
  unset -f command 2>/dev/null
  command -v java
)" || :
JAVACCMD="$(
  set +e
  unset -f command 2>/dev/null
  command -v javac
)" || :

Impact

With the quotes, the shell literally looks for executables called set, unset, and command, which do not exist. This causes the command substitution to fail and JAVACMD/JAVACCMD remain empty. As a consequence, the Java-based download fallback will never be selected even when wget/curl are missing.

References

• PR: chore: update Maven Wrapper to 3.9.9 #185
• Comment: chore: update Maven Wrapper to 3.9.9 #185 (comment)

[nanotaboada]

These scripts were generated using mvn -N wrapper:wrapper -Dmaven=3.9.9
Any issues or oddities should be fixed upstream in the Maven Wrapper project.