DO NOT REPLY [Bug 28865] New: - JDepend does not accept jars, but can with minor change.

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=28865>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28865

JDepend <classespath> does not accept jars, but can with minor change.

           Summary: JDepend <classespath> does not accept jars, but can with
                    minor change.
           Product: Ant
           Version: 1.6.1
          Platform: All
        OS/Version: Other
            Status: NEW
          Severity: Enhancement
          Priority: Other
         Component: Optional Tasks
        AssignedTo: dev@ant.apache.org
        ReportedBy: carl_peasnell@hotmail.com


JDepend versions >=2.6 can accept .jar, .war and .zip files containing classes 
for analysis.

The current JDependTask has code in methods executeAsForked and executeInVM 
that prevents this by incorrectly preempting JDepend failing in code such as:
  if (!f.exists() || !f.isDirectory()) {
                    String msg = "\""
                        + f.getPath()
                        + "\" does not represent a valid"
                        + " directory. JDepend would fail.";
                    log(msg);
                    throw new BuildException(msg);
  }

If the "|| !f.isDirectory()" conditions are removed the code works with .jar 
files.  I have tested this with both fork="yes" and fork="no".

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org