Issue Details (XML | Word | Printable)

Key: MPDASHBOARD-31
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Carlos Sanchez
Reporter: Carlos Sanchez
Votes: 2
Watchers: 4
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Dashboard Plugin

Create aggregators for Cobertura

Created: 11/Aug/05 03:18 PM   Updated: 26/Jan/06 12:42 PM   Resolved: 30/Nov/05 12:30 PM
Return to search
Component/s: None
Affects Version/s: None
Fix Version/s: 1.9

Time Tracking:
Not Specified


Brice Copy added a comment - 18/Nov/05 02:40 AM

Unfortunately, the Cobertura aggregators are already in the Dashboard plugin official docs.
Any chance of seeing this happening soon ? The current version in Subversion is broken, it returns :

/users/cruise/.maven/cache/maven-dashboard-plugin-1.9-SNAPSHOT/plugin.jelly:210:33: <j:import> could not import script

Is there a patch for this one can apply ?


Carlos Sanchez added a comment - 18/Nov/05 02:49 AM

I believe everything is already right. What properties are you using?


pkernevez added a comment - 30/Nov/05 11:34 AM

I think I found the pb (I had the same error when I run a mutlitarget:site goal)):
The file target\docs\cobertura\coverage.xml start with the TDT declaration
<!DOCTYPE coverage SYSTEM "http://cobertura.sourceforge.net/xml/coverage-02.dtd">

When 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 tdt.

I am behind a proxy that refuse any internet connection.

I will try to find a solution.

The full error when I call dashboard:report-single is:
BUILD FAILED
File...... C:\maven\home\.maven\cache\maven-dashboard-plugin-1.9-SNAPSHOT\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


Carlos Sanchez added a comment - 30/Nov/05 12:30 PM

I think this is already in place, other problems should be covered in different issues


Nascif A. Abousalh-Neto added a comment - 25/Jan/06 05:22 PM

I just downloaded the 1.8 version of the plugin to get Cobertura support and got this error:

BUILD FAILED
File...... C:\Documents and Settings\naabou\.maven\cache\maven-multiproject-plugin-1.4.1\plugin.jelly
Element... maven:reactor
Line...... 104
Column.... 9
Unable to obtain goal [dashboard:report-single] – C:\Documents and Settings\naabou\.maven\cache\maven-dashboard-plugin-1.8\plugin.jelly:192:23: <a:fail> Invalid aggregator name [coberturalipc]

I checked the .jar and and there are no aggregators for Cobertura there, although thery are already showing in the official documentation.

Any idea why? From the discussion above they should be available, right?

Thanks,
Nascif


Lukas Theussl added a comment - 25/Jan/06 05:30 PM

As you see in the fix version for this issue, the Cobertura aggregators have only been implemented in version 1.9 of the dashboard plugin. This one is not published yet (but is expected to come out maybe next week ).


Lukas Theussl added a comment - 25/Jan/06 05:35 PM

Actually, if you want to do some testing: there is a SNAPSHOT available:

maven plugin:download -Dmaven.repo.remote=http://www.ibiblio.org/maven,http://cvs.apache.org/repository/ -DgroupId=maven -DartifactId=maven-dashboard-plugin -Dversion=1.9-SNAPSHOT

Let us know how it works!


Nascif A. Abousalh-Neto added a comment - 26/Jan/06 09:16 AM

Thanks, I am playing with it. I found two problems so far:

1) The Cobertura Aggregator It introduces dependencies on the latest Jelly library (u:loadText). This is an issue for Maven 1.0.2 users like me and should be documented or removed - there are ways to skip the external DTD parsing by configuring a SAX parser, see http://www-128.ibm.com/developerworks/xml/library/x-tipcfsx.html

2) Somehow this latest code is breaking the JXR plugin. More specifically, it resets the variable ${plugin-resources} to c:/. I am still investigating that. I had to update the Jelly libraries due to the problem above so it might be related.

Regards,
Nascif


Lukas Theussl added a comment - 26/Jan/06 11:12 AM

1) I think the problem is not u:loadText but rather u:replace, which does not work with maven 1.0.2. I have re-published a snapshot where I'm using the commons-lang StringUtils for the string replacement, can you check if that helps in any way?

2) I don't see how the jxr plugin is connected with this. Any more details?


Nascif A. Abousalh-Neto added a comment - 26/Jan/06 11:21 AM

Ah, it is bad. I have to change jxr, multiproject and dasboard to provide hard-coded references to the plugin.resources variable - it was broken in all of them.

I think that updating the Jelly libraries might have been a bad idea. Is it possible for you to test 1.9 with a pure 1.0.2 and make sure it works? I tried Maven 1.1 but it broke so many of my plugins that I gave up on it.

Now the build runs, multiproject completes, but no dashboard html output is generated. And I have no error messages this time, I have no idea what is going on. Is there a new target for the 1.9 version? I am calling maven with the multiproject:site target, as I did before for the 1.7 version, is there a new target now to force the dashboard generation?

I can send you a maven log if you think it might help.

Thanks,
Nascif


Lukas Theussl added a comment - 26/Jan/06 11:33 AM

With jelly you are always in for a good surprise !

I can try to test but I need some time since I'm not using cobertura, so I was hoping you could help out. Are you subsribed to the maven dev or user list? It would be easier to discuss that there.


Nascif A. Abousalh-Neto added a comment - 26/Jan/06 12:42 PM

OK, just subscribed to dev, replying on the list.
Thanks,
Nascif