On Sun, May 05, 2002 at 05:01:13AM +0200, St�phane MOR wrote:
> Why not ...  Maybe would it be easier to strip out prefixes in
> checkstyle.dvsl, using string utils or something. (I don't know the
> o.a.commons.strings package, so I can't tell what is possible ...) The
> problem is that one can't know what is the name of the parent
> directory.  Something like $path.Parent could do it (I don't know if
> that is present in any tools available to checkstyle.dvsl )

The problem here is that it becomes a hack.  In order to properly
normalize the filenames that checkstyle uses to the jxr output, you'd
have to basically check each filename and stip off any prefix that
matches any of the <sourceDirectories> specified in the POM (project
descriptor - aka project.xml).

> The creation of a wrapper is probably a too big workaround for a
> simple "bug" as this one.

Well, we actually did this for the JXR task (not a wrapper, but modified
the JXR task directly); however, I was not in favor of that either.  I'd
much rather prefer to see only a single source directory being specified
which would eliminate all of these normalization issues that we
currently face.  That way all of these Ant tasks that process the source
directory can use the same base directory as a starting point which will
result in an output is consistent.

Reply via email to