[THRIFT-2843] Automake configure.ac has possible typo related to Java

• Type: Bug
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 0.9.3
• Component/s: Build Process
• Assignee: Cash Costello
• Reporter: Cash Costello
• Created: 19/Nov/14 20:24
• Updated: 24/Nov/14 21:10
• Resolved: 21/Nov/14 15:08

Description

Looks like it tests for the java compiler twice. I'm guessing the second test is unnecessary or should be a test for the Java runtime.

AX_THRIFT_LIB(java, [Java], yes)
if test "$with_java" = "yes";  then
  AX_JAVAC_AND_JAVA
  AC_PATH_PROG([ANT], [ant])
  AX_CHECK_ANT_VERSION($ANT, 1.7)
  AC_SUBST(CLASSPATH)
  AC_SUBST(ANT_FLAGS)
  if test "x$JAVAC" != "x" && test "x$JAVAC" != "x" && test "x$ANT" != "x" ; then
    have_java="yes"
  fi
fi