[
https://jira.codehaus.org/browse/MNG-5658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=358630#comment-358630
]
Jonathan Sailor commented on MNG-5658:
--------------------------------------
This patch should not have been landed as-is. The change "remove $( ) around
the absolute file name." is not correct; /usr/libexec/java_home needs to be run
to find the correct value for $JAVA_HOME, such as
/Library/Java/JavaVirtualMachines/jdk1.8.0_25.jdk/Contents/Home, which is what
the syntax $(blah) does. This change sets $JAVA_HOME to the path
/usr/libexec/java_home itself, which is incorrect.
> Syntax error in bin/mvn on Solaris SPARC
> ----------------------------------------
>
> Key: MNG-5658
> URL: https://jira.codehaus.org/browse/MNG-5658
> Project: Maven
> Issue Type: Bug
> Affects Versions: 3.2.2
> Environment: Solaris SPARC 10 with Oracle JDK 1.7.0_60
> Reporter: Frank Langelage
> Assignee: Jason van Zyl
> Fix For: 3.2.3
>
> Attachments: maven_bin_mvn.patch
>
>
> The latest addition to bin/mvn breaks mvn running on Solaris.
> Syntax error in line 86.
> The Bourne shell /bin/sh does not understand this syntax with the brackets.
> if [[ -z "$JAVA_HOME" && -x /usr/libexec/java_home ]] ; then
> #
> # Apple JDKs
> #
> export JAVA_HOME=$(/usr/libexec/java_home)
> fi
> ;;
> changing the line with the assignment to
> export JAVA_HOME=/usr/libexec/java_home
> like the lines before makes it running again.
--
This message was sent by Atlassian JIRA
(v6.1.6#6162)
