From 1ffd296062dc16611d068df9e90164e4b3ef2143 Mon Sep 17 00:00:00 2001
From: kkolinko <kkolinko@13f79535-47bb-0310-9956-ffa450edef68>
Date: Wed, 10 Feb 2010 13:10:51 +0000
Subject: [PATCH] Improvements for building jasper-jdt.jar: - Rebuild
 jasper-jdt.jar if jdt.jar was updated or redownloaded, or the build file was
 updated - Purge target directory before unjarring, to avoid repacking stale
 files - Clean unjarred files after the build as they are not needed

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@908484 13f79535-47bb-0310-9956-ffa450edef68
---
 build.xml | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/build.xml b/build.xml
index fd20af64b..38710a74d 100644
--- a/build.xml
+++ b/build.xml
@@ -1664,13 +1664,14 @@ Apache Tomcat ${version} native binaries for Win64 AMD64/EMT64 platform.
       <param name="destfile" value="${jdt.jar}"/>
       <param name="destdir" value="${jdt.home}"/>
     </antcall>
-    <mkdir dir="${jasper-jdt.home}"/>
     <condition property="no.build.jasper-jdt">
-      <available file="${jasper-jdt.jar}"/>
+      <and>
+        <available file="${jasper-jdt.jar}"/>
+        <uptodate targetfile="${jasper-jdt.jar}" srcfile="${jdt.jar}"/>
+        <uptodate targetfile="${jasper-jdt.jar}" srcfile="${basedir}/build.xml"/>
+      </and>
     </condition>
-    <antcall target="build-jasper-jdt">
-      <param name="basedir" value="${jasper-jdt.home}" />
-    </antcall>
+    <antcall target="build-jasper-jdt" />
 
   </target>
 
@@ -1764,6 +1765,8 @@ Apache Tomcat ${version} native binaries for Win64 AMD64/EMT64 platform.
 
   <target name="build-jasper-jdt" depends="build-manifests"
   	      unless="no.build.jasper-jdt">
+    <delete dir="${jasper-jdt.home}"/>
+    <mkdir dir="${jasper-jdt.home}"/>
     <unjar src="${jdt.jar}" dest="${jasper-jdt.home}" />
   	<jarIt jarfile="${jasper-jdt.jar}"
   	  filesDir="${jasper-jdt.home}"
@@ -1771,6 +1774,12 @@ Apache Tomcat ${version} native binaries for Win64 AMD64/EMT64 platform.
   	  manifest="${tomcat.manifests}/default.manifest"
   	  notice="${tomcat.home}/res/META-INF/jasper-jdt.jar.notice"
   	  license="${tomcat.home}/res/META-INF/jasper-jdt.jar.license"/>
+    <delete includeEmptyDirs="true">
+      <fileset dir="${jasper-jdt.home}">
+        <include name="**/*"/>
+        <exclude name="jasper-jdt.jar"/>
+      </fileset>
+    </delete>
   </target>
 
 
-- 
2.11.0