RE : RE : Checkstyle report problem
We agreed recently that relative paths would be allowed in the POM, and made absolute as part of the POM processing. -- dIon Gillard, Multitask Consulting Blog: http://blogs.codehaus.org/people/dion/ "Nicolas FRANK" <[EMAIL PROTECTED]> wrote on 20/08/2003 09:00:27 PM: > So the documentation should be updated (I was about to send the same link but > you did it faster ;-) )... > I changed my project.xml source directory and all is ok with the cvs head > version. In my understand, the ${basedir} requirement is ok as it allows more > fexibility (for example, for migration purpose). But the documentation should > be very clear about that as this is not the natural expected behaviour. > > Thank's. > > -Message d'origine- > De : Simon Matic Langford [mailto:[EMAIL PROTECTED] > Envoyé : mercredi 20 août 2003 12:23 > À : 'Maven Users List' > Objet : RE: RE : Checkstyle report problem > > > 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 > > ${basedir}/src/java > > > > ----- 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 > value="${pom.build.sourceDirectory}"/> while it should be > 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: > > > > > > > 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 t
RE : RE : Checkstyle report problem
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 value="${pom.build.sourceDirectory}"/> while it should be 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: > > > > 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 > > > > > > > > > > > > > > > > > > plicationException.java > > > > > > > > > > > > 1 > > > > > > > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > > > > > > > opicHandler.java > > > > > > > > > > > > 55 > > > > > > > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl > > > : > > > > > > > > > > > > > > > > > > > > > > > >> > value="${file.attribute('name').getValue()}"/> > > > > > >> > > > > value="${name.substring(mavenTool.toInteger(srcDirLength.toString()))}"/> > > > > > >> > newChar="/"/> > > > > > > > > > > > >> > select="count($file/error)"/> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ${name} > > > > > > > > > > > >> > pattern="0"/> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > - > > 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] >
RE: RE : Checkstyle report problem
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 > > ${basedir}/src/java > > > > - 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 > value="${pom.build.sourceDirectory}"/> while it should be > 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: > > > > > > > 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.: > > > > > > > > > > > > >
Re: RE : Checkstyle report problem
I updated the docs in cvs. - Original Message - From: "Simon Matic Langford" <[EMAIL PROTECTED]> To: "'Maven Users List'" <[EMAIL PROTECTED]> Sent: Wednesday, August 20, 2003 12:23 PM Subject: RE: RE : Checkstyle report problem 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 > ${basedir}/src/java > > - 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 value="${pom.build.sourceDirectory}"/> while it should be 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: > > > > 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 > > > > > > > > > > > > > > > > > > plicationException.java > > > > > > > > > > > > 1 > > > > > > > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > > > > > > > opicHandler.java > > > > > > > > > > > > 55 > > > > > > > > > > > > > > > > > > I am not totally sure, but it may something to
RE : RE : Checkstyle report problem
So the documentation should be updated (I was about to send the same link but you did it faster ;-) )... I changed my project.xml source directory and all is ok with the cvs head version. In my understand, the ${basedir} requirement is ok as it allows more fexibility (for example, for migration purpose). But the documentation should be very clear about that as this is not the natural expected behaviour. Thank's. -Message d'origine- De : Simon Matic Langford [mailto:[EMAIL PROTECTED] Envoyé : mercredi 20 août 2003 12:23 À : 'Maven Users List' Objet : RE: RE : Checkstyle report problem 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 > ${basedir}/src/java > > - 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 value="${pom.build.sourceDirectory}"/> while it should be 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: > > > > 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 > > > > > > > > > > > > > > > > > > plicationException.java > > > > > > > > > > > > 1 > > > >
RE: RE : Checkstyle report problem
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 > ${basedir}/src/java > > - 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 value="${pom.build.sourceDirectory}"/> while it should be 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: > > > > 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 > > > > > > > > > > > > > > > > > > plicationException.java > > > > > > > > > > > > 1 > > > > > > > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > > > > > > > opicHandler.java > > > > > > > > > > > > 55 > > > > > > > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl > > > : > > > > > > > > > > > > > > > > > > > > > > >
Re: RE : Checkstyle report problem
No, because we recommend to use ${basedir} un pom declaration ${basedir}/src/java - 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 while it should be -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: > > 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 > > > > > > > > > > > > plicationException.java > > > > > > > > 1 > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > opicHandler.java > > > > > > > > 55 > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl > > : > > > > > > > > > > > > > > > >> value="${file.attribute('name').getValue()}"/> > > > >> > value="${name.substring(mavenTool.toInteger(srcDirLength.toString()))}"/> > > > >> newChar="/"/> > > > > > > > >> select="count($file/error)"/> > > > > > > > > > > > > > > > > > > > > ${name} > > > > > > > >> pattern="0"/> > > > > > > > > > > > > > > > > > > > > - > 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]
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 while it should be -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: > > 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 > > > > > > > > > > > > plicationException.java > > > > > > > > 1 > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > opicHandler.java > > > > > > > > 55 > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl > > : > > > > > > > > > > > > > > > >> value="${file.attribute('name').getValue()}"/> > > > >> > value="${name.substring(mavenTool.toInteger(srcDirLength.toString()))}"/> > > > >> newChar="/"/> > > > > > > > >> select="count($file/error)"/> > > > > > > > > > > > > > > > > > > > > ${name} > > > > > > > >> pattern="0"/> > > > > > > > > > > > > > > > > > > > > - > 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]
RE : RE : Checkstyle report problem
Thank's a lot... It does the job ! -Message d'origine- De : Simon Matic Langford [mailto:[EMAIL PROTECTED] Envoyé : mardi 19 août 2003 11:38 À : 'Maven Users List' Objet : RE: RE : Checkstyle report problem 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 < --- > 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: > > 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 > > > > > > > > > > > > plicationException.java > > > > > > > > 1 > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > opicHandler.java > > > > > > > > 55 > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl: > > > > > > > > > > > > > > > >> value="${file.attribute('name').getValue()}"/> > > > >> > value="${name.substring(mavenTool.toInteger(srcDirLength.toStr > ing()))}"/> > > > >> newChar="/"/> > > > > > > > >> select="count($file/error)"/> > > > > > > > > > > > > > > > > > > > > ${name} > > > > > > > >> pattern="0"/> > > > > > > > > > > > > > > > > > > > > - > 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] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
RE: RE : Checkstyle report problem
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 < --- > 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: > > 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 > > > > > > > > > > > > plicationException.java > > > > > > > > 1 > > > > > > > > > > > > TopiHandler becomes: > > > > > > > > > > > > opicHandler.java > > > > > > > > 55 > > > > > > > > > > > > I am not totally sure, but it may something to do with this in > > > maven-checkstyle-plugin-2.0-SNAPSHOT\plugin-resources\checkstyle.jsl: > > > > > > > > > > > > > > > >> value="${file.attribute('name').getValue()}"/> > > > >> > value="${name.substring(mavenTool.toInteger(srcDirLength.toStr > ing()))}"/> > > > >> newChar="/"/> > > > > > > > >> select="count($file/error)"/> > > > > > > > > > > > > > > > > > > > > ${name} > > > > > > > >> pattern="0"/> > > > > > > > > > > > > > > > > > > > > - > 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]