[MNG-5829] mvn shell script fails with syntax error on Solaris 10

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.3
• Fix Version/s: 3.5.0-alpha-1, 3.5.0
• Component/s: Command Line
• Environment: Solaris 10
• Assignee: Michael Osipov
• Reporter: Zac Thompson
• Created: 22/May/15 18:51
• Updated: 28/Mar/20 15:00
• Resolved: 30/Jan/17 12:00

Description

The $(cmd) idiom is used extensively in bin/mvn from lines 199-220, but this is not supported by older incarnations of /bin/sh (used on the shebang line). The `cmd` idiom is slightly more portable, and looks like it can probably be directly substituted.

find_maven_basedir() {
  local basedir=$(pwd)
  local wdir=$(pwd)
  while [ "$wdir" != '/' ] ; do
    wdir=$(cd "$wdir/.."; pwd)
    if [ -d "$wdir"/.mvn ] ; then
      basedir=$wdir
      break
    fi
  done
  echo "${basedir}"
}

# concatenates all lines of a file
concat_lines() {
  if [ -f "$1" ]; then
    echo "$(tr -s '\n' ' ' < "$1")"
  fi
}

export MAVEN_PROJECTBASEDIR=${MAVEN_BASEDIR:-$(find_maven_basedir)}
MAVEN_OPTS="$(concat_lines "$MAVEN_PROJECTBASEDIR/.mvn/jvm.config") $MAVEN_OPTS"

See MNG-5658 for an earlier case of a similar problem.

Issue Links

• is duplicated by: MNG-6291 Error: Could not find or load main class $(concat_lines
• is related to: MNG-6195 Use consistent quoting forms in mvn launcher script
• supercedes: MNG-5837 Syntax error in bin/mvn on Solaris SPARC