Stefan, this one is a weird situation IMHO.... I'm not
sure where Javadoc was complaining, but the way the
tests were structured, they were looking at
"src/etc/testcases/taskdefs/javadoc/java" as the
source directory, while the test Java file there was
in the package etc.testcases.taskdefs.javadoc.java,
making its true source directory "src". Pulling
package "*" from
src/etc/testcases/taskdefs/javadoc/java was returning
the java directory itself, aka "" and caused the Java
file to be omitted in testPathelementPath. My
temporary solution was to change the source directory
to ${root}/src just to quiet "Gumpster." But this
doesn't _really_ test the package "*" functionality;
we may need to place an actual package structure
beneath javadoc/java to (a) keep it self-contained and
(b) test adequately.
WDYT (all)?
-Matt
--- [EMAIL PROTECTED] wrote:
> Author: bodewig
> Date: Tue Feb 14 21:27:55 2006
> New Revision: 377942
>
> URL:
> http://svn.apache.org/viewcvs?rev=377942&view=rev
> Log:
> javadoc doesn't like the palckage name '', skip it
>
> Modified:
>
>
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
>
> Modified:
>
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
> URL:
>
http://svn.apache.org/viewcvs/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java?rev=377942&r1=377941&r2=377942&view=diff
>
==============================================================================
> ---
>
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
> (original)
> +++
>
ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
> Tue Feb 14 21:27:55 2006
> @@ -2136,12 +2136,20 @@
> });
>
> if (files.length > 0) {
> + if ("".equals(dirs[i])) {
> + log(baseDir
> + + " contains source
> files in the default package,"
> + + " you must specify
> them as source files"
> + + " not packages.",
> + Project.MSG_WARN);
> + } else {
> containsPackages = true;
> String packageName =
>
> dirs[i].replace(File.separatorChar, '.');
> if
> (!addedPackages.contains(packageName)) {
>
> addedPackages.addElement(packageName);
> pn.addElement(packageName);
> + }
> }
> }
> }
>
>
>
>
---------------------------------------------------------------------
> To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> For additional commands, e-mail:
> [EMAIL PROTECTED]
>
>
__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
http://mail.yahoo.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
