No, because we recommend to use ${basedir} un pom declaration
<sourceDirectory>${basedir}/src/java</sourceDirectory>----- Original Message ----- From: "Nicolas FRANK" <[EMAIL PROTECTED]> To: "Maven Users List" <[EMAIL PROTECTED]> Sent: Wednesday, August 20, 2003 10:43 AM Subject: RE : RE : Checkstyle report problem Yes, the cvs head (http://cvs.apache.org/viewcvs/maven/src/plugins-build/checkstyle/src/plugin - resources/checkstyle.jsl?rev=HEAD&content-type=text/vnd.viewcvs-markup) of checkstyle.jsl gives <j:set var="fullSrcDir" value="${pom.build.sourceDirectory}"/> while it should be <j:set var="fullSrcDir" value="${basedir}/${pom.build.sourceDirectory}"/> -----Message d'origine----- De : [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Envoyé : mercredi 20 août 2003 01:48 À : Maven Users List Objet : RE : Checkstyle report problem I was sure I fixed this recently. Are you sure this happens with the new HEAD? -- dIon Gillard, Multitask Consulting Blog: http://blogs.codehaus.org/people/dion/ "Nicolas FRANK" <[EMAIL PROTECTED]> wrote on 19/08/2003 07:31:11 PM: > 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\UserContextAcuf.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.toString()))}"/> > > > > <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] _____________________________________________________________________ Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France _____________________________________________________________________ Envie de discuter en "live" avec vos amis ? Télécharger MSN Messenger http://www.ifrance.com/_reloc/m la 1ère messagerie instantanée de France --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
