I posted a small patch (hack) that fixes the problem a few days ago - by
changing the checkstyle.dvsl
It removes the preceding 5 chars from the links generated. ie java/ and test/.
Think your solution sounds better than the hack I submitted. My solution only
works for subdirs of src with 4 chars (java and test are ok)
Vincent Massol wrote:
> Hi,
>
> The checkstyle task uses ${maven.src.dir} as its fileset. The problem
> with this are :
>
> 1/ If you have the following directory structure :
>
> src
> |_ java
> |_ test
>
> then the checkstyle reports give file name prefixed with java/ or test/
>
> but more importantly
>
> 2/ it breaks the xref link (on the line number) as xref uses the
> maven.src.set sourcepathref (which I imagine come from the POM) to
> generate its html files.
>
> Solution:
>
> Use maven.src.set in the checkstyle task. However, we need to transform
> maven.src.set into a patternset to be able to use it in a fileset ...
>
> A solution I can see is to write an Ant task that does this. Something
> like :
>
> <path2filelist patternsetid="new pattern set id">
> <path ...>
> </path2filelist>
>
> where <path...> accepts a pathid attribute referring to an already
> defined path set.
>
> I'm sure there are other ways (especially with Ant 1.5). Do you know any
> other ?
>
> Thanks
> -Vincent
>
>
> --
> To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
