From bf5fd7938002272a1857de51dbe181ea5257c265 Mon Sep 17 00:00:00 2001
From: Stefan Bodewig <bodewig@apache.org>
Date: Wed, 26 Nov 2008 05:34:32 +0000
Subject: [PATCH] fail early if stylesheet doesn't exist.  PR 34525.

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@720746 13f79535-47bb-0310-9956-ffa450edef68
---
 WHATSNEW                                                | 3 +++
 src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java | 6 ++++++
 src/tests/antunit/taskdefs/xslt-test.xml                | 7 +++++++
 3 files changed, 16 insertions(+)

diff --git a/WHATSNEW b/WHATSNEW
index e90ba4611..6965369b2 100644
--- a/WHATSNEW
+++ b/WHATSNEW
@@ -548,6 +548,9 @@ Other changes:
    -docfilessubdirs and -excludedocfilessubdir command line arguments.
    Bugzilla Report 34455.
 
+ * <xslt> now fails early if a specified stylesheet doesn't exist.
+   Bugzilla Report 34525.
+
 Changes from Ant 1.7.0 TO Ant 1.7.1
 =============================================
 
diff --git a/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java b/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java
index f11627c4d..bfb059abc 100644
--- a/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java
+++ b/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java
@@ -322,6 +322,12 @@ public class XSLTProcess extends MatchingTask implements XSLTLogger {
             } else {
                 styleResource = xslResource;
             }
+
+            if (!styleResource.isExists()) {
+                throw new BuildException("stylesheet " + styleResource
+                                         + " doesn't exist.");
+            }
+
             // if we have an in file and out then process them
             if (inFile != null && outFile != null) {
                 process(inFile, outFile, styleResource);
diff --git a/src/tests/antunit/taskdefs/xslt-test.xml b/src/tests/antunit/taskdefs/xslt-test.xml
index 8d1c12c69..cd9bc12fc 100644
--- a/src/tests/antunit/taskdefs/xslt-test.xml
+++ b/src/tests/antunit/taskdefs/xslt-test.xml
@@ -66,4 +66,11 @@ undefined='<xsl:value-of select="$undefined"/>'
        value="set='somevalue'"/>
   </target>
 
+  <target name="testStyleDoesntExist" depends="setUp">
+    <au:expectfailure expectedmessage="i-m-not-there.xslt doesn't exist.">
+      <xslt in="${legacy.dir}/data.xml"
+            out="${output}/out.xml"
+            style="i-m-not-there.xslt"/>
+    </au:expectfailure>
+  </target>
 </project>