Yes it could be a good idea. commons-lang 2.0 is already used in several plugins : http://maven.apache.org/maven-1.x/reference/plugins/dependency-convergence-report.html#commons-lang:commons-lang
Arnaud > -----Message d'origine----- > De : Lukas Theussl [mailto:[EMAIL PROTECTED] > Envoyé : vendredi 2 décembre 2005 18:29 > À : Maven Developers List > Objet : Re: [jira] Commented: (MPDASHBOARD-34) Cobertura > aggregator don't support offline mode > > Arnaud, > > I noticed that I have introduced the same incompatibility > (util:replace a string) at several places in the xdoc plugin. > > Would it be a valid workaround to put commons-lang as a > dependency and use the static StringUtils.replace instead? > > Cheers, > -Lukas > > > Arnaud Heritier (JIRA) wrote: > > > [ > > > http://jira.codehaus.org/browse/MPDASHBOARD-34?page=comments#action_52 > > 591 ] > > > >Arnaud Heritier commented on MPDASHBOARD-34: > >-------------------------------------------- > > > >your patch will make the plugin compatible only with maven 1.1 the > >jelly tag util:replace bundled in maven 1.0.2 doesn't allow > to replace a String. > >Personnaly I'm not against to create plugins compatible only > with maven 1.1 if we don't have a workaround. > > > > > > > >>Cobertura aggregator don't support offline mode > >>----------------------------------------------- > >> > >> Key: MPDASHBOARD-34 > >> URL: http://jira.codehaus.org/browse/MPDASHBOARD-34 > >> Project: maven-dashboard-plugin > >> Type: Bug > >> Versions: 1.9 > >> Reporter: pkernevez > >> Attachments: MPDASHBOARD-34.patch > >> > >> > >>We I throw the goal "dashboard:report-single" I had an > error if I am off line: > >>BUILD FAILED > >>File...... > >>C:\maven\home\.maven\cache\maven-dashboard-plugin-1.9-SNAPSH > OT\plugin. > >>jelly > >>Element... j:import > >>Line...... 210 > >>Column.... -1 > >>file:/C:/maven/home/.maven/cache/maven-dashboard-plugin-1.9- > SNAPSHOT/plugin-resources/aggregators/coberturaloc.jelly:32: > >>-1: <x:parse> Error on line 2 of document : External entity > not found: > >>"http://cobertura.sourceforge.net/xml/coverage-0 > >>2.dtd". Nested exception: External entity not found: > "http://cobertura.sourceforge.net/xml/coverage-02.dtd";. > >>Total time : 2 seconds > >>Finished at : Wednesday, November 30, 2005 6:25:22 PM CET > This error > >>is due to the DTD declaration into the file > "target\docs\cobertura\coverage.xml": > >><!DOCTYPE coverage SYSTEM > >>"http://cobertura.sourceforge.net/xml/coverage-02.dtd";> > >>If your remove this declaration the target > dashboard:report-single succed in. > >>This is because the plugin need to be on line to be able to > download the dtd. > >>I tried to change the aggretors' jelly script but the jelly > attribute "validate" doesn't seems to perform: > >>I replace the part: > >> <j:when test="${artifactAsFile.exists()}"> > >> <x:parse var="doc" xml="${artifactAsFile}"/> > >> <x:expr > select="count($doc/coverage/packages/package/classes/class/lin > es/[EMAIL PROTECTED]>0])"/> > >> </j:when> > >>with : > >> <j:when test="${artifactAsFile.exists()}"> > >> <x:parse var="doc" xml="${artifactAsFile}" validate="false"/> > >> <x:expr > select="count($doc/coverage/packages/package/classes/class/lin > es/[EMAIL PROTECTED]>0])"/> > >> </j:when> > >>But I don't resolve my issue > >> > >> > > > > > > > > > --------------------------------------------------------------------- > 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]
