In CVS head, pom.build.sourceDirectory should always return an absolute path. -- dIon Gillard, Multitask Consulting Blog: http://blogs.codehaus.org/people/dion/
"Nicolas FRANK" <[EMAIL PROTECTED]> wrote on 20/08/2003 06:43:46 PM: > 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] >
