From fc9b7af57212bdbc5b2b250db7e7a90ea7125fc1 Mon Sep 17 00:00:00 2001
From: "Chris M. Hostetter" <hossman@apache.org>
Date: Fri, 5 Jan 2007 23:14:26 +0000
Subject: [PATCH] refactoring some redundency with the junit check, also
 stealing the more useful failure message from lucene-java's common-build.xml

git-svn-id: https://svn.apache.org/repos/asf/incubator/solr/trunk@493229 13f79535-47bb-0310-9956-ffa450edef68
---
 build.xml | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/build.xml b/build.xml
index 7cfbac10d01..4eb6aaa12bc 100644
--- a/build.xml
+++ b/build.xml
@@ -96,6 +96,8 @@
   <property name="junit.output.dir" location="${dest}/test-results"/>
   <property name="junit.reports" location="${dest}/test-results/reports"/>
   <property name="junit.includes" value="**/Test*.java,**/*Test.java"/>
+  <available property="junitPresent"
+             classname="junit.framework.TestCase" />
 
   <!-- Macro for building checksum files
        This is only needed until the "format" option is supported
@@ -162,8 +164,6 @@
   <target name="compile"
           description="Compile the source code."
           depends="checkJunitPresence">
-    <fail unless="junitPresent"
-          message="Couldn't find JUnit on runtime classpath." />
 
     <mkdir dir="${dest}" />
 
@@ -239,18 +239,21 @@
   </path>
 
   <!-- Check that JUnit is present. -->
-  <target name="checkJunitPresence"
-          description="Check that JUnit is present.">
-    <available property="junitPresent"
-               classname="junit.framework.TestCase" />
+  <target name="checkJunitPresence">
+    <!-- no description so it doesn't show up in -projecthelp -->  
+    <fail unless="junitPresent"> 
+      ##################################################################
+      JUnit not found.
+      Please make sure junit.jar is in ANT_HOME/lib, or made available
+      to Ant using other mechanisms like -lib or CLASSPATH.
+      ##################################################################
+    </fail>
   </target>
 
   <!-- Compile unit tests. -->
   <target name="compileTests"
           description="Compile unit tests."
           depends="compile,checkJunitPresence">
-    <fail unless="junitPresent"
-          message="Couldn't find JUnit on the runtime classpath." />
 
     <mkdir dir="${dest}/tests" />
     <javac