Author: peterreilly
Date: Mon Sep  4 15:47:46 2006
New Revision: 440200

URL: http://svn.apache.org/viewvc?view=rev&rev=440200
Log:
Bugzilla report 28865: allow files in classespath for <jdepend>

Modified:
    ant/core/trunk/WHATSNEW
    
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java

Modified: ant/core/trunk/WHATSNEW
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/WHATSNEW?view=diff&rev=440200&r1=440199&r2=440200
==============================================================================
--- ant/core/trunk/WHATSNEW (original)
+++ ant/core/trunk/WHATSNEW Mon Sep  4 15:47:46 2006
@@ -26,6 +26,8 @@
   the language jars to be specified in the build script.
   Bugzilla report 29676.
 * trim the driver attribute on the <sql> task. Bugzilla report 21228.
+* Allow (jar) files as well as directories to be given to jdepend.
+  Bugzilla report 28865.
 
 Changes from Ant 1.6.5 to Ant 1.7.0Beta1
 ========================================

Modified: 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java?view=diff&rev=440200&r1=440199&r2=440200
==============================================================================
--- 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java
 (original)
+++ 
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java
 Mon Sep  4 15:47:46 2006
@@ -473,18 +473,17 @@
             if (getClassespath() != null) {
                 // This is the new, better way - use classespath instead
                 // of sourcespath.  The code is currently the same - you
-                // need class files in a directory to use this - jar files
-                // coming soon....
+                // need class files in a directory to use this or jar files.
                 String[] cP = getClassespath().list();
                 for (int i = 0; i < cP.length; i++) {
                     File f = new File(cP[i]);
                     // not necessary as JDepend would fail, but why loose
                     // some time?
-                    if (!f.exists() || !f.isDirectory()) {
+                    if (!f.exists()) {
                         String msg = "\""
                             + f.getPath()
                             + "\" does not represent a valid"
-                            + " directory. JDepend would fail.";
+                            + " file or directory. JDepend would fail.";
                         log(msg);
                         throw new BuildException(msg);
                     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to