Correct. CVS Head should now do it automatically. -- dIon Gillard, Multitask Consulting Blog: http://blogs.codehaus.org/people/dion/
"Simon Matic Langford" <[EMAIL PROTECTED]> wrote on 20/08/2003 08:23:21 PM: > not according to your documentation: > > http://maven.apache.org/start/integrate.html > > 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: Emmanuel Venisse [mailto:[EMAIL PROTECTED] > > Sent: 20 August 2003 10:26 > > To: Maven Users List > > Subject: Re: RE : Checkstyle report problem > > > > > > 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/checkst > > yle/src/plugin > > - > > resources/checkstyle.jsl?rev=HEAD&content-type=text/vnd.viewcv > > s-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\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] > > > > _____________________________________________________________________ > > 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] > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] >
