I've fixed this locally on beta-10, by changing checkstyle.jsl in the
checkstyle plugin, but
we have our own customised plugin which contains some changes I've
submitted to jira, so
I thought it might be us.
if anyone wants to apply it and it isn't already, then here's the patch:
59c59
< <j:set var="fullSrcDir"
value="${pom.build.sourceDirectory}"/>
---
> <j:set var="fullSrcDir"
value="${basedir}/${pom.build.sourceDirectory}"/>
s
The information contained in this e-mail is intended only for the person
or
entity to which it is addressed and may contain confidential and/or
privileged material. If You are not the intended recipient of this
e-mail,
the use of this information or any disclosure, copying or distribution
is
Prohibited and may be unlawful. If you received this in error, please
contact the sender and delete the material from any computer. The views
expressed in this e-mail may not necessarily be the views of The PCMS
Group
plc and should not be taken as authority to carry out any instruction
contained.
> -----Original Message-----
> From: Nicolas FRANK [mailto:[EMAIL PROTECTED]
> Sent: 19 August 2003 10:31
> To: Maven Users List
> Subject: RE : Checkstyle report problem
>
>
> I have something similar (but this is not the same problem),
> with the cvs
> head of the checkstyle plugin.
>
> The problem happens only when using the reactor:
>
> <m:reactor basedir="${basedir}" includes="**/project.xml"
> goals="site:deploy"
> banner="deployement site web" ignoreFailures="false"/>
>
> With the reactor the report links look like that:
>
> xref\ier\src\java\com\financeactive\acuf\canvasDoc\xml\UserCon
> textAcuf.html#2
> 8
>
> The correct version is well created when directly running
> site:deploy on the
> ejb-tier project:
>
> xref/com/financeactive/acuf/canvasDoc/xml/UserContextAcuf.html#28
>
> I guess this has something related with the sourceDir, but
> couldn't find any
> way to fix it...
> Any idea (I am sure that other people have already deal with that)?
>
>
> -----Message d'origine-----
> De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> Envoy� : mardi 19 ao�t 2003 04:07
> � : Maven Users List
> Objet : Re: Checkstyle report problem
>
>
> Is this from beta-10 or from the stable branch. I believe
> this was fixed
> after b10 was released.
> --
> dIon Gillard, Multitask Consulting
> Blog: http://blogs.codehaus.org/people/dion/
>
>
> Boris Ekelchik <[EMAIL PROTECTED]> wrote on
> 19/08/2003
> 11:39:06 AM:
>
> > Generated links and labels in the checkstyle report seam to loose
> > random number of characters from original class names, e.g.:
> >
> >
> >
> > Checkstyle report row for ApplicationException is
> >
> > <tr>
> >
> > <td>
> >
> > <a href="#plicationException.java">plicationException.java</a>
> >
> > </td>
> >
> > <td>1</td>
> >
> > </tr>
> >
> >
> >
> > TopiHandler becomes:
> >
> > <tr>
> >
> > <td>
> >
> > <a href="#opicHandler.java">opicHandler.java</a>
> >
> > </td>
> >
> > <td>55</td>
> >
> > </tr>
> >
> >
> >
> > I am not totally sure, but it may something to do with this in
> >
> maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl:
> >
> >
> >
> > <j:forEach var="file" items="${files}">
> >
> > <!-- Type coercion doesn't work worth a fuck in jexl.
> > -->
> >
> > <j:set var="name"
> > value="${file.attribute('name').getValue()}"/>
> >
> > <j:set var="name"
> >
> value="${name.substring(mavenTool.toInteger(srcDirLength.toStr
> ing()))}"/>
> >
> > <util:replace var="name" value="${name}" oldChar="\\"
> > newChar="/"/>
> >
> > <!--- +1 is for the trailing slash above -->
> >
> > <j:set var="errorCount"><x:expr
> > select="count($file/error)"/></j:set>
> >
> >
> >
> > <j:if test="${errorCount != 0}">
> >
> > <tr>
> >
> > <td>
> >
> > <a href="#${name}">${name}</a>
> >
> > </td>
> >
> > <td><doc:formatAsNumber string="${errorCount}"
> > pattern="0"/></td>
> >
> > </tr>
> >
> > </j:if>
> >
> > </j:forEach>
> >
> > </table>
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
