How about we just make the path -> filelist happen in the maven classes
that set the properties up, and add this as a new property. This bit one of
my guys yesterday, so I agree this needs to be done.
--
dIon Gillard, Multitask Consulting
Work: http://www.multitask.com.au
Developers: http://adslgateway.multitask.com.au/developers
"Vincent
Massol" To:
<[EMAIL PROTECTED]>
<vmassol@octo. cc:
com> Subject: Checkstyle problem
05/23/02 08:58
PM
Please respond
to "Turbine
Maven
Developers
List"
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]>
