Maven 1.x Dashboard Plugin

Create aggregators for Cobertura

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.9
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Activity

Hide
Brice Copy added a comment -

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 ?

Show
Brice Copy added a comment - 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 ?
Hide
Carlos Sanchez added a comment -

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

Show
Carlos Sanchez added a comment - I believe everything is already right. What properties are you using?
Hide
pkernevez added a comment -

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

Show
pkernevez added a comment - 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
Hide
Carlos Sanchez added a comment -

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

Show
Carlos Sanchez added a comment - I think this is already in place, other problems should be covered in different issues
Hide
Nascif A. Abousalh-Neto added a comment -

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

Show
Nascif A. Abousalh-Neto added a comment - 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
Hide
Lukas Theussl added a comment -

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 ).

Show
Lukas Theussl added a comment - 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 ).
Hide
Lukas Theussl added a comment -

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!

Show
Lukas Theussl added a comment - 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!
Hide
Nascif A. Abousalh-Neto added a comment -

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

Show
Nascif A. Abousalh-Neto added a comment - 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
Hide
Lukas Theussl added a comment -

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?

Show
Lukas Theussl added a comment - 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?
Hide
Nascif A. Abousalh-Neto added a comment -

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

Show
Nascif A. Abousalh-Neto added a comment - 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
Hide
Lukas Theussl added a comment -

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.

Show
Lukas Theussl added a comment - 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.
Hide
Nascif A. Abousalh-Neto added a comment -

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

Show
Nascif A. Abousalh-Neto added a comment - OK, just subscribed to dev, replying on the list. Thanks, Nascif

People

Vote (2)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: